X-Git-Url: https://git.argeo.org/?a=blobdiff_plain;f=org.argeo.cms%2Fsrc%2Forg%2Fargeo%2Fcms%2Fauth%2FHttpSessionLoginModule.java;h=1bbe359b9f8b478a6d986e75f32d47717e42873b;hb=6f20d8f8fb510fd7fbecfafd0254126624a5e9e1;hp=7622eef27da26c49f13d6a56ef63a68e27f1387e;hpb=e89299764ff15d485c94849d5b6e283d7502dac5;p=lgpl%2Fargeo-commons.git

diff --git a/org.argeo.cms/src/org/argeo/cms/auth/HttpSessionLoginModule.java b/org.argeo.cms/src/org/argeo/cms/auth/HttpSessionLoginModule.java
index 7622eef27..1bbe359b9 100644
--- a/org.argeo.cms/src/org/argeo/cms/auth/HttpSessionLoginModule.java
+++ b/org.argeo.cms/src/org/argeo/cms/auth/HttpSessionLoginModule.java
@@ -80,9 +80,8 @@ public class HttpSessionLoginModule implements LoginModule {
 				httpSession = request.getSession(true);
 			}
 			String httpSessionId = httpSession.getId();
-			// authorization = (Authorization)
-			// request.getSession().getAttribute(HttpContext.AUTHORIZATION);
-			// if (authorization == null) {
+			if (log.isTraceEnabled())
+				log.trace("HTTP login: " + request.getPathInfo() + " #" + httpSessionId);
 			Collection<ServiceReference<CmsSession>> sr;
 			try {
 				sr = bc.getServiceReferences(CmsSession.class,
@@ -108,8 +107,12 @@ public class HttpSessionLoginModule implements LoginModule {
 		extractHttpAuth(request);
 		extractClientCertificate(request);
 		if (authorization == null) {
+			if (log.isTraceEnabled())
+				log.trace("HTTP login: " + false);
 			return false;
 		} else {
+			if (log.isTraceEnabled())
+				log.trace("HTTP login: " + true);
 			return true;
 		}
 	}
@@ -211,7 +214,7 @@ public class HttpSessionLoginModule implements LoginModule {
 			String certDn = req.getHeader(clientDnHttpHeader);
 			// TODO retrieve more cf. https://httpd.apache.org/docs/current/mod/mod_ssl.html
 			// String issuerDn = req.getHeader("SSL_CLIENT_I_DN");
-			if (certDn != null && !certDn.trim().equals("")) {
+			if (certDn != null && !certDn.trim().equals("(null)")) {
 				sharedState.put(CmsAuthUtils.SHARED_STATE_NAME, certDn);
 				sharedState.put(CmsAuthUtils.SHARED_STATE_CERTIFICATE_CHAIN, "");
 				if (log.isDebugEnabled())