Yair Zaslavsky has posted comments on this change.

Change subject: aaa: Fix group name issues for legacy provider
......................................................................


Patch Set 3: Code-Review-1

(1 comment)

http://gerrit.ovirt.org/#/c/32652/3/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/utils/aaa/AuthzUtils.java
File 
backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/utils/aaa/AuthzUtils.java:

Line 3: import java.util.Collection;
Line 4: 
Line 5: public class AuthzUtils {
Line 6: 
Line 7:     public static String getAuthzNameFromEntityName(String entityName, 
Collection<String> authzProvidersNames) {
still need to fix this.
Line 8:         return entityName != null  && entityName.contains("@")? 
entityName.substring(entityName.lastIndexOf('@') + 1) : null;
Line 9:     }
Line 10: 
Line 11: 


-- 
To view, visit http://gerrit.ovirt.org/32652
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I8c1feed08e33156c794b230fa04f33f2ebd82240
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yair Zaslavsky <yzasl...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to