Tomáš Došek has posted comments on this change.
Change subject: engine: Fix for BZ#967890 - enhance logging of JndiAction level
in dEBUG mode
......................................................................
Patch Set 1: (2 inline comments)
....................................................
File
backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/kerberos/JndiAction.java
Line 112: }
Line 113: userGuid.append(guid);
Line 114: log.debug("User guid is: " +
userGuid.toString());
Line 115: return AuthenticationResult.OK;
Line 116: }
Here I'd go for another bug. and and separate patch. I'll file one and insert
the patch provided in my earlier comment.
Line 117: // Print user GUID only if it was not printed
previously in while loop
Line 118: if (!answer.hasMoreElements()) {
Line 119: log.debug("User guid is: " +
userGuid.toString());
Line 120: }
Line 116: }
Line 117: // Print user GUID only if it was not printed
previously in while loop
Line 118: if (!answer.hasMoreElements()) {
Line 119: log.debug("User guid is: " +
userGuid.toString());
Line 120: }
In the second case current logic of JndiAction jumps to "No user information
found" hence I guess the null value for user ID is correct, because it's the
one JndAction class picked up.
Line 121: // Print also URI and complete query path
Line 122: log.debug("URI is: " + uri.toString());
Line 123: log.debug("Complete query path is: " +
currentLdapServer);
Line 124: System.out.println("No user in Directory was
found for " + userName
--
To view, visit http://gerrit.ovirt.org/15176
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I84525767c98eda9eba522dc5518c02c10c55fbfa
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomáš Došek <[email protected]>
Gerrit-Reviewer: Martin Peřina <[email protected]>
Gerrit-Reviewer: Tomáš Došek <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
