Yair Zaslavsky has posted comments on this change.

Change subject: engine: Fix for BZ#967890 - enhance logging of JndiAction level 
in dEBUG mode
......................................................................


Patch Set 1: (1 inline comment)

....................................................
File 
backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/kerberos/JndiAction.java
Line 119:                        log.debug("User guid is: " + 
userGuid.toString());
Line 120:                     }
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: " + 
ldapQueryPath.toString());
I agree with Laszlo's comment. Worth adding as well.
Line 124:                     System.out.println("No user in Directory was 
found for " + userName
Line 125:                             + ". Trying next LDAP server in list");
Line 126:                 } else {
Line 127:                     System.out.println(InstallerConstants.ERROR_PREFIX


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia6c2504ba7d6f77492862e6ccf6c0770fdab0fd7
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomáš Došek <[email protected]>
Gerrit-Reviewer: Laszlo Hornyak <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to