Bug 1104107 – [RFE][AAA] support only UPN format for usernames

Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.

Bug 1104107 - [RFE][AAA] support only UPN format for usernames

Summary:	[RFE][AAA] support only UPN format for usernames

Status:	CLOSED CURRENTRELEASE

Aliases:	None

Product:	ovirt-engine
Classification:	oVirt
Component:	AAA (Show other bugs)
Sub Component:
Version:	3.5.0
Hardware:	Unspecified Unspecified

Priority	high Severity low (vote)
Target Milestone:	ovirt-4.0.0-alpha
Target Release:	4.0.0
Assigned To:	Ravi Nori
QA Contact:	Gonza
Docs Contact:

URL:
Whiteboard:
Keywords:	FutureFeature, Reopened

Depends On:	ovirt-aaa-sso
Blocks:	oVirt-AAA-rewrite
	Show dependency tree / graph

Reported:	2014-06-03 06:00 EDT by Yair Zaslavsky
Modified:	2016-07-15 01:38 EDT (History)
CC List:	14 users (show)

See Also:
Fixed In Version:	ovirt 4.0.0 alpha1
Doc Type:	Enhancement
Doc Text:	The SAM user format of profile\user is no longer supported.
Story Points:	---
Clone Of:
Environment:
Last Closed:	2016-07-05 03:52:16 EDT
Type:	Bug
Regression:	---
Mount Type:	---
Documentation:	---
CRM:
Verified Versions:
Category:	---
oVirt Team:	Infra
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---

Flags:	rule-engine: ovirt‑4.0.0+ pnovotny: testing_plan_complete+ rule-engine: planning_ack+ alonbl: devel_ack+ pstehlik: testing_ack+

Attachments	(Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

Groups: None (edit)

Description Yair Zaslavsky 2014-06-03 06:00:06 EDT

Description of problem:

From looking at REST-API code I saw there is support (probably back from 2.2 days) for domain\user format.
This support should probably be removed.


Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 1 Yair Zaslavsky 2014-07-26 09:06:48 EDT

This collides with BZ1123545.
Turns out that is still still used.
So we should reach a decision whether to support non UPN formats or not.

If we we do want to fix  PROFILE\user format, the fix should be around

BasicAutnenticationFilter.java

And also check the web.xml of the rest-api - web-app -


 <filter>
    <filter-name>BasicAuthenticationFilter</filter-name>
    <filter-class>org.ovirt.engine.core.aaa.filters.BasicAuthenticationFilter</filter-class>
    <init-param>
      <param-name>user-name-format</param-name>
      <param-value>UPN</param-value>
    </init-param>
  </filter>

Comment 4 Alon Bar-Lev 2015-11-17 07:43:04 EST

In 4.0 we can just drop this, no need to be backward compatible.

Comment 5 Alon Bar-Lev 2015-12-01 04:13:07 EST

Ravi, this should be trivial.

Comment 6 Ravi Nori 2015-12-01 10:22:28 EST

Fixed as part of SSO work

Comment 7 Red Hat Bugzilla Rules Engine 2015-12-01 10:26:00 EST

Fixed bug tickets must have version flags set prior to fixing them. Please set the correct version flags and move the bugs back to the previous status after this is corrected.

Comment 8 Red Hat Bugzilla Rules Engine 2015-12-01 10:26:00 EST

Bug tickets must have version flags set prior to targeting them to a release. Please ask maintainer to set the correct version flags and only then set the target milestone.

Comment 9 Red Hat Bugzilla Rules Engine 2015-12-01 10:26:00 EST

Target release should be placed once a package build is known to fix a issue. Since this bug is not modified, the target version has been reset. Please use target milestone to plan a fix for a oVirt release.

Comment 10 Red Hat Bugzilla Rules Engine 2015-12-01 10:27:50 EST

Target release should be placed once a package build is known to fix a issue. Since this bug is not modified, the target version has been reset. Please use target milestone to plan a fix for a oVirt release.

Comment 11 Red Hat Bugzilla Rules Engine 2015-12-01 10:27:50 EST

This request has been proposed for two releases. This is invalid flag usage. The ovirt-future release flag has been cleared. If you wish to change the release flag, you must clear one release flag and then set the other release flag to ?.

Comment 12 Gonza 2016-06-15 05:08:06 EDT

Verified with:
ovirt-engine-4.1.0-0.0.master.20160605051229.gitb398fa2.el7.centos.noarch

Comment 13 Sandro Bonazzola 2016-07-05 03:52:16 EDT

oVirt 4.0.0 has been released, closing current release.

Note You need to log in before you can comment on or make changes to this bug.