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: Ia6c2504ba7d6f77492862e6ccf6c0770fdab0fd7
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/39/15139/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..5a6d445 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.hasMorElements() != true) {
+                       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: " + 
ldapQueryPath.toString());
                     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/15139
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia6c2504ba7d6f77492862e6ccf6c0770fdab0fd7
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