[java-idp-plugin-jetty] branch main updated: JJETTY-42 - Jetty CLI inits its logger too early

[Codeberg]

This is an automated email from the git hooks/post-receive script.

codeberg pushed a commit to branch main
in repository java-idp-plugin-jetty.

View the commit online:
https://codeberg.org/Shibboleth/java-idp-plugin-jetty/commit/8c34f4c0074c1d654a2e2c8978deba3d18c15fdf

The following commit(s) were added to refs/heads/main by this push:
     new 8c34f4c  JJETTY-42 - Jetty CLI inits its logger too early
8c34f4c is described below

commit 8c34f4c0074c1d654a2e2c8978deba3d18c15fdf
Author: Scott Cantor <scott at restingparrotsoftware.com>
AuthorDate: Mon Apr 20 12:07:16 2026 -0400

    JJETTY-42 - Jetty CLI inits its logger too early
    
    https://shibboleth.atlassian.net/browse/JJETTY-42
    
    Defer class logger.
---
 .../idp/plugin/jetty/cli/impl/JettyDownloadCLI.java         | 13 ++++++++++---
 1 file changed, 10 insertions(+), 3 deletions(-)

diff --git a/jetty-cli/src/main/java/net/shibboleth/idp/plugin/jetty/cli/impl/JettyDownloadCLI.java b/jetty-cli/src/main/java/net/shibboleth/idp/plugin/jetty/cli/impl/JettyDownloadCLI.java
index 9da9e03..6b7722c 100644
--- a/jetty-cli/src/main/java/net/shibboleth/idp/plugin/jetty/cli/impl/JettyDownloadCLI.java
+++ b/jetty-cli/src/main/java/net/shibboleth/idp/plugin/jetty/cli/impl/JettyDownloadCLI.java
@@ -52,10 +52,13 @@ import net.shibboleth.shared.logic.Constraint;
 import net.shibboleth.shared.primitive.LoggerFactory;
 import net.shibboleth.shared.spring.httpclient.resource.HTTPResource;
 
+/**
+ * CLI class for Jetty downloader tool.
+ */
 public class JettyDownloadCLI extends AbstractIdPHomeAwareCommandLine<JettyDownloadArguments> {
 
     /** Logger. */
-    @Nullable private final Logger log = LoggerFactory.getLogger(JettyDownloadCLI.class);
+    @Nullable private Logger log;
 
     /** the keystore (if provided). */
     @Nullable private Path keyStore;
@@ -87,8 +90,12 @@ public class JettyDownloadCLI extends AbstractIdPHomeAwareCommandLine<JettyDownl
 
     /** {@inheritDoc} */
     @Override
-    protected Logger getLogger() {
-        return log;
+    @Nonnull protected Logger getLogger() {
+        Logger theLog = log;
+        if (theLog == null) {
+            theLog = log = LoggerFactory.getLogger(JettyDownloadCLI.class);
+        }
+        return theLog;
     }
 
     /** {@inheritDoc} */

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.