From: Mathieu Baudier <mbaudier@argeo.org>
Date: Wed, 13 Dec 2023 09:39:48 +0000 (+0100)
Subject: Improve logging
X-Git-Tag: v2.3.24~2
X-Git-Url: http://git.argeo.org/?p=lgpl%2Fargeo-commons.git;a=commitdiff_plain;h=46fe088ea0839d61df1450c0bea48b203411e028

Improve logging
---

diff --git a/org.argeo.cms/src/org/argeo/cms/auth/UserAdminLoginModule.java b/org.argeo.cms/src/org/argeo/cms/auth/UserAdminLoginModule.java
index 47b36f446..638547d24 100644
--- a/org.argeo.cms/src/org/argeo/cms/auth/UserAdminLoginModule.java
+++ b/org.argeo.cms/src/org/argeo/cms/auth/UserAdminLoginModule.java
@@ -283,13 +283,15 @@ public class UserAdminLoginModule implements LoginModule {
 		if (log.isDebugEnabled()) {
 			StringBuilder msg = new StringBuilder();
 			msg.append("Logged in to CMS: '" + authorization + "' (" + authorization.getName() + ")\n");
-			for (Principal principal : subject.getPrincipals()) {
-				msg.append("  Principal: " + principal.getName()).append(" (")
-						.append(principal.getClass().getSimpleName()).append(")\n");
-			}
-			for (Object credential : subject.getPublicCredentials()) {
-				msg.append("  Public Credential: " + credential).append(" (")
-						.append(credential.getClass().getSimpleName()).append(")\n");
+			if (log.isTraceEnabled()) {
+				for (Principal principal : subject.getPrincipals()) {
+					msg.append("  Principal: " + principal.getName()).append(" (")
+							.append(principal.getClass().getSimpleName()).append(")\n");
+				}
+				for (Object credential : subject.getPublicCredentials()) {
+					msg.append("  Public Credential: " + credential).append(" (")
+							.append(credential.getClass().getSimpleName()).append(")\n");
+				}
 			}
 			log.debug(msg);
 		}