Tomáš Došek has uploaded a new change for review.

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

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

Enhanced logging possibility for JndiAction class in debug mode.
This enables user to view more detailed info on what query path, URI and user
was used to perform the authentication and ticket creation.

Change-Id: I84525767c98eda9eba522dc5518c02c10c55fbfa
Bug-Url: https://bugzilla.redhat.com/967890
Signed-off-by: Tomas Dosek <[email protected]>
---
M 
backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/kerberos/JndiAction.java
1 file changed, 7 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/76/15176/1

diff --git 
a/backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/kerberos/JndiAction.java
 
b/backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/kerberos/JndiAction.java
index af42b2d..9ad0541 100644
--- 
a/backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/kerberos/JndiAction.java
+++ 
b/backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/kerberos/JndiAction.java
@@ -114,7 +114,13 @@
                         log.debug("User guid is: " + userGuid.toString());
                         return AuthenticationResult.OK;
                     }
-
+                    // Print user GUID only if it was not printed previously 
in while loop
+                    if (!answer.hasMoreElements()) {
+                       log.debug("User guid is: " + userGuid.toString());
+                    }
+                    // Print also URI and complete query path
+                    log.debug("URI is: " + uri.toString());
+                    log.debug("Complete query path is: " + currentLdapServer);
                     System.out.println("No user in Directory was found for " + 
userName
                             + ". Trying next LDAP server in list");
                 } else {


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

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

Reply via email to