Switch to jboss-logmanager

Drop the old `jboss-logmanager-embedded` in favor of `jboss-logmanager`.
quarkusio · Jul 24, 2023 · 2be7359 · 2be7359
1 parent 9acf766
commit 2be7359
Show file tree

Hide file tree

Showing 31 changed files with 126 additions and 155 deletions.
diff --git a/bom/application/pom.xml b/bom/application/pom.xml
@@ -166,7 +166,7 @@
         <dekorate.version>3.7.0</dekorate.version> <!-- Please check with Java Operator SDK team before updating -->
         <maven-invoker.version>3.2.0</maven-invoker.version>
         <awaitility.version>4.2.0</awaitility.version>
-        <jboss-logmanager.version>1.1.1</jboss-logmanager.version>
+        <jboss-logmanager.version>3.0.0.Final</jboss-logmanager.version>
         <flyway.version>9.21.0</flyway.version>
         <yasson.version>3.0.3</yasson.version>
         <liquibase.version>4.20.0</liquibase.version>
@@ -3229,31 +3229,31 @@
                 <version>${commons-logging-jboss-logging.version}</version>
             </dependency>
             <dependency>
+                <!-- NOTE: this dependency exists in order to help users who are using the old GAV for logging -->
+                <!-- The version doesn't need to be a property since it will be locked at 1.2.0.Final forever -->
                 <groupId>org.jboss.logmanager</groupId>
                 <artifactId>jboss-logmanager-embedded</artifactId>
+                <version>1.2.0.Final</version>
+            </dependency>
+            <dependency>
+                <groupId>org.jboss.logmanager</groupId>
+                <artifactId>jboss-logmanager</artifactId>
                 <version>${jboss-logmanager.version}</version>
                 <exclusions>
                     <exclusion>
                         <groupId>org.jboss.modules</groupId>
                         <artifactId>jboss-modules</artifactId>
                     </exclusion>
                     <exclusion>
-                        <groupId>org.glassfish</groupId>
-                        <artifactId>javax.json</artifactId>
+                        <groupId>org.eclipse.parsson</groupId>
+                        <artifactId>jakarta.json</artifactId>
                     </exclusion>
                 </exclusions>
             </dependency>
             <dependency>
                 <groupId>org.jboss.logmanager</groupId>
                 <artifactId>log4j2-jboss-logmanager</artifactId>
                 <version>${log4j2-jboss-logmanager.version}</version>
-                <exclusions>
-                    <!-- we use the embedded version -->
-                    <exclusion>
-                        <groupId>org.jboss.logmanager</groupId>
-                        <artifactId>jboss-logmanager</artifactId>
-                    </exclusion>
-                </exclusions>
             </dependency>
             <!--
             While we are not affected by CVE-2021-4428 as we are only using the Log4j2 API,
@@ -3268,13 +3268,6 @@
                 <groupId>org.jboss.logmanager</groupId>
                 <artifactId>log4j-jboss-logmanager</artifactId>
                 <version>${log4j-jboss-logmanager.version}</version>
-                <exclusions>
-                    <!-- we use the embedded version -->
-                    <exclusion>
-                        <groupId>org.jboss.logmanager</groupId>
-                        <artifactId>jboss-logmanager</artifactId>
-                    </exclusion>
-                </exclusions>
             </dependency>
 
             <dependency>

diff --git a/core/builder/pom.xml b/core/builder/pom.xml
@@ -40,7 +40,7 @@
 
         <dependency>
             <groupId>org.jboss.logmanager</groupId>
-            <artifactId>jboss-logmanager-embedded</artifactId>
+            <artifactId>jboss-logmanager</artifactId>
             <scope>test</scope>
         </dependency>
 

diff --git a/core/deployment/src/main/java/io/quarkus/deployment/logging/LoggingResourceProcessor.java b/core/deployment/src/main/java/io/quarkus/deployment/logging/LoggingResourceProcessor.java
@@ -38,7 +38,7 @@
 import org.jboss.jandex.DotName;
 import org.jboss.jandex.IndexView;
 import org.jboss.logging.Logger;
-import org.jboss.logmanager.EmbeddedConfigurator;
+import org.jboss.logmanager.LogContextInitializer;
 import org.jboss.logmanager.LogManager;
 import org.objectweb.asm.Opcodes;
 
@@ -217,7 +217,7 @@ void miscSetup(
         systemProp
                 .accept(new NativeImageSystemPropertyBuildItem("java.util.logging.manager", "org.jboss.logmanager.LogManager"));
         provider.accept(
-                new ServiceProviderBuildItem(EmbeddedConfigurator.class.getName(), InitialConfigurator.class.getName()));
+                new ServiceProviderBuildItem(LogContextInitializer.class.getName(), InitialConfigurator.class.getName()));
     }
 
     @BuildStep

diff --git a/core/processor/pom.xml b/core/processor/pom.xml
@@ -46,7 +46,7 @@
         </dependency>
         <dependency>
             <groupId>org.jboss.logmanager</groupId> 
-            <artifactId>jboss-logmanager-embedded</artifactId> 
+            <artifactId>jboss-logmanager</artifactId>
             <scope>test</scope> 
         </dependency>
     </dependencies>

diff --git a/core/runtime/pom.xml b/core/runtime/pom.xml
@@ -59,7 +59,7 @@
         </dependency>
         <dependency>
             <groupId>org.jboss.logmanager</groupId>
-            <artifactId>jboss-logmanager-embedded</artifactId>
+            <artifactId>jboss-logmanager</artifactId>
         </dependency>
         <dependency>
             <groupId>org.jboss.logging</groupId>
@@ -148,7 +148,13 @@
                     </lesserPriorityArtifacts>
                     <parentFirstArtifacts>
                         <parentFirstArtifact>io.quarkus:quarkus-bootstrap-runner</parentFirstArtifact>
+                        <parentFirstArtifact>io.smallrye.common:smallrye-common-constraint</parentFirstArtifact>
+                        <parentFirstArtifact>io.smallrye.common:smallrye-common-cpu</parentFirstArtifact>
+                        <parentFirstArtifact>io.smallrye.common:smallrye-common-expression</parentFirstArtifact>
                         <parentFirstArtifact>io.smallrye.common:smallrye-common-io</parentFirstArtifact>
+                        <parentFirstArtifact>io.smallrye.common:smallrye-common-net</parentFirstArtifact>
+                        <parentFirstArtifact>io.smallrye.common:smallrye-common-os</parentFirstArtifact>
+                        <parentFirstArtifact>io.smallrye.common:smallrye-common-ref</parentFirstArtifact>
                         <parentFirstArtifact>org.wildfly.common:wildfly-common</parentFirstArtifact>
                         <parentFirstArtifact>org.graalvm.sdk:graal-sdk</parentFirstArtifact>
                         <parentFirstArtifact>org.graalvm.js:js</parentFirstArtifact>
@@ -162,7 +168,7 @@
                         <parentFirstArtifact>io.quarkus:quarkus-bootstrap-maven-resolver</parentFirstArtifact>
                         <parentFirstArtifact>org.slf4j:slf4j-api</parentFirstArtifact>
                         <parentFirstArtifact>org.jboss.slf4j:slf4j-jboss-logmanager</parentFirstArtifact>
-                        <parentFirstArtifact>org.jboss.logmanager:jboss-logmanager-embedded</parentFirstArtifact>
+                        <parentFirstArtifact>org.jboss.logmanager:jboss-logmanager</parentFirstArtifact>
                         <parentFirstArtifact>org.jboss.logging:jboss-logging</parentFirstArtifact>
                         <parentFirstArtifact>org.apache.maven:maven-model</parentFirstArtifact>
                         <parentFirstArtifact>org.apache.maven.resolver:maven-resolver-api</parentFirstArtifact>
@@ -208,7 +214,7 @@
                         <runnerParentFirstArtifact>io.quarkus:quarkus-development-mode-spi</runnerParentFirstArtifact>
                         <!-- logging dependencies always need to be loaded by the JDK ClassLoader -->
                         <runnerParentFirstArtifact>org.jboss.logging:jboss-logging</runnerParentFirstArtifact>
-                        <runnerParentFirstArtifact>org.jboss.logmanager:jboss-logmanager-embedded</runnerParentFirstArtifact>
+                        <runnerParentFirstArtifact>org.jboss.logmanager:jboss-logmanager</runnerParentFirstArtifact>
                         <!-- This is needed as it's a dependency of the JBoss Logging so it too needs to be loaded by the JDK ClassLoader -->
                         <runnerParentFirstArtifact>org.wildfly.common:wildfly-common</runnerParentFirstArtifact>
                         <!-- This is needed because it contains some jar handling classes -->

diff --git a/core/runtime/src/main/java/io/quarkus/runtime/logging/BannerFormatter.java b/core/runtime/src/main/java/io/quarkus/runtime/logging/BannerFormatter.java
diff --git a/core/runtime/src/main/java/io/quarkus/runtime/logging/LoggingSetupRecorder.java b/core/runtime/src/main/java/io/quarkus/runtime/logging/LoggingSetupRecorder.java
@@ -26,13 +26,15 @@
 import java.util.logging.LogManager;
 import java.util.logging.LogRecord;
 
-import org.jboss.logmanager.EmbeddedConfigurator;
+import org.jboss.logmanager.ExtFormatter;
 import org.jboss.logmanager.LogContext;
+import org.jboss.logmanager.LogContextInitializer;
 import org.jboss.logmanager.Logger;
 import org.jboss.logmanager.errormanager.OnlyOnceErrorManager;
 import org.jboss.logmanager.filters.AllFilter;
 import org.jboss.logmanager.formatters.ColorPatternFormatter;
 import org.jboss.logmanager.formatters.PatternFormatter;
+import org.jboss.logmanager.formatters.TextBannerFormatter;
 import org.jboss.logmanager.handlers.AsyncHandler;
 import org.jboss.logmanager.handlers.ConsoleHandler;
 import org.jboss.logmanager.handlers.FileHandler;
@@ -266,7 +268,7 @@ public void accept(String categoryName, CategoryConfig config) {
         }
         addNamedHandlersToRootHandlers(config.handlers, namedHandlers, handlers, errorManager);
         InitialConfigurator.DELAYED_HANDLER.setAutoFlush(false);
-        InitialConfigurator.DELAYED_HANDLER.setHandlers(handlers.toArray(EmbeddedConfigurator.NO_HANDLERS));
+        InitialConfigurator.DELAYED_HANDLER.setHandlers(handlers.toArray(LogContextInitializer.NO_HANDLERS));
         return shutdownNotifier;
     }
 
@@ -362,7 +364,7 @@ public static void initializeBuildTimeLogging(LogConfig config, LogBuildTimeConf
         }
         addNamedHandlersToRootHandlers(config.handlers, namedHandlers, handlers, errorManager);
         InitialConfigurator.DELAYED_HANDLER.setAutoFlush(false);
-        InitialConfigurator.DELAYED_HANDLER.setBuildTimeHandlers(handlers.toArray(EmbeddedConfigurator.NO_HANDLERS));
+        InitialConfigurator.DELAYED_HANDLER.setBuildTimeHandlers(handlers.toArray(LogContextInitializer.NO_HANDLERS));
     }
 
     private boolean shouldCreateNamedHandlers(LogConfig logConfig,
@@ -556,21 +558,13 @@ private static Handler configureConsoleHandler(final ConsoleConfig config,
                 bannerSupplier = possibleBannerSupplier.getValue().get();
             }
             if (ColorSupport.isColorEnabled(consoleRuntimeConfig, config)) {
+                formatter = new ColorPatternFormatter(config.darken, config.format);
                 color = true;
-                ColorPatternFormatter colorPatternFormatter = new ColorPatternFormatter(config.darken,
-                        config.format);
-                if (bannerSupplier != null) {
-                    formatter = new BannerFormatter(colorPatternFormatter, true, bannerSupplier);
-                } else {
-                    formatter = colorPatternFormatter;
-                }
             } else {
-                PatternFormatter patternFormatter = new PatternFormatter(config.format);
-                if (bannerSupplier != null) {
-                    formatter = new BannerFormatter(patternFormatter, false, bannerSupplier);
-                } else {
-                    formatter = patternFormatter;
-                }
+                formatter = new PatternFormatter(config.format);
+            }
+            if (bannerSupplier != null) {
+                formatter = new TextBannerFormatter(bannerSupplier, ExtFormatter.wrap(formatter, false));
             }
         }
         final ConsoleHandler consoleHandler = new ConsoleHandler(

diff --git a/docs/src/main/asciidoc/class-loading-reference.adoc b/docs/src/main/asciidoc/class-loading-reference.adoc
@@ -150,7 +150,7 @@ or a `LinkageError` can occur.
         <parentFirstArtifacts>
             <parentFirstArtifact>io.quarkus:quarkus-bootstrap-core</parentFirstArtifact>
             <parentFirstArtifact>io.quarkus:quarkus-development-mode-spi</parentFirstArtifact>
-            <parentFirstArtifact>org.jboss.logmanager:jboss-logmanager-embedded</parentFirstArtifact>
+            <parentFirstArtifact>org.jboss.logmanager:jboss-logmanager</parentFirstArtifact>
             <parentFirstArtifact>org.jboss.logging:jboss-logging</parentFirstArtifact>
             <parentFirstArtifact>org.ow2.asm:asm</parentFirstArtifact>
         </parentFirstArtifacts>

diff --git a/extensions/datasource/common/pom.xml b/extensions/datasource/common/pom.xml
@@ -18,8 +18,8 @@
             <scope>test</scope>
         </dependency>
         <dependency>
-            <groupId>org.jboss.logmanager</groupId> 
-            <artifactId>jboss-logmanager-embedded</artifactId> 
+            <groupId>org.jboss.logmanager</groupId>
+            <artifactId>jboss-logmanager</artifactId>
             <scope>test</scope> 
         </dependency>
     </dependencies>

diff --git a/extensions/devservices/db2/pom.xml b/extensions/devservices/db2/pom.xml
@@ -40,8 +40,8 @@
             <scope>test</scope>
         </dependency>
         <dependency>
-            <groupId>org.jboss.logmanager</groupId> 
-            <artifactId>jboss-logmanager-embedded</artifactId> 
+            <groupId>org.jboss.logmanager</groupId>
+            <artifactId>jboss-logmanager</artifactId>
             <scope>test</scope> 
         </dependency>
     </dependencies>

diff --git a/extensions/devservices/derby/pom.xml b/extensions/devservices/derby/pom.xml
@@ -30,9 +30,9 @@
             <scope>test</scope>
         </dependency>
         <dependency>
-            <groupId>org.jboss.logmanager</groupId> 
-            <artifactId>jboss-logmanager-embedded</artifactId> 
-            <scope>test</scope> 
+            <groupId>org.jboss.logmanager</groupId>
+            <artifactId>jboss-logmanager</artifactId>
+            <scope>test</scope>
         </dependency>
     </dependencies>
     <build>

diff --git a/extensions/devservices/h2/pom.xml b/extensions/devservices/h2/pom.xml
@@ -26,9 +26,9 @@
             <scope>test</scope>
         </dependency>
         <dependency>
-            <groupId>org.jboss.logmanager</groupId> 
-            <artifactId>jboss-logmanager-embedded</artifactId> 
-            <scope>test</scope> 
+            <groupId>org.jboss.logmanager</groupId>
+            <artifactId>jboss-logmanager</artifactId>
+            <scope>test</scope>
         </dependency>
     </dependencies>
     <build>

diff --git a/extensions/devservices/mariadb/pom.xml b/extensions/devservices/mariadb/pom.xml
@@ -40,9 +40,9 @@
             <scope>test</scope>
         </dependency>
         <dependency>
-            <groupId>org.jboss.logmanager</groupId> 
-            <artifactId>jboss-logmanager-embedded</artifactId> 
-            <scope>test</scope> 
+            <groupId>org.jboss.logmanager</groupId>
+            <artifactId>jboss-logmanager</artifactId>
+            <scope>test</scope>
         </dependency>
     </dependencies>
     <build>

diff --git a/extensions/devservices/mssql/pom.xml b/extensions/devservices/mssql/pom.xml
@@ -40,9 +40,9 @@
             <scope>test</scope>
         </dependency>
         <dependency>
-            <groupId>org.jboss.logmanager</groupId> 
-            <artifactId>jboss-logmanager-embedded</artifactId> 
-            <scope>test</scope> 
+            <groupId>org.jboss.logmanager</groupId>
+            <artifactId>jboss-logmanager</artifactId>
+            <scope>test</scope>
         </dependency>
     </dependencies>
     <build>

diff --git a/extensions/devservices/mysql/pom.xml b/extensions/devservices/mysql/pom.xml
@@ -36,9 +36,9 @@
             <scope>test</scope>
         </dependency>
         <dependency>
-            <groupId>org.jboss.logmanager</groupId> 
-            <artifactId>jboss-logmanager-embedded</artifactId> 
-            <scope>test</scope> 
+            <groupId>org.jboss.logmanager</groupId>
+            <artifactId>jboss-logmanager</artifactId>
+            <scope>test</scope>
         </dependency>
         <dependency>
             <groupId>mysql</groupId>

diff --git a/extensions/devservices/oracle/pom.xml b/extensions/devservices/oracle/pom.xml
@@ -41,7 +41,7 @@
         </dependency>
         <dependency>
             <groupId>org.jboss.logmanager</groupId>
-            <artifactId>jboss-logmanager-embedded</artifactId>
+            <artifactId>jboss-logmanager</artifactId>
             <scope>test</scope>
         </dependency>
     </dependencies>

diff --git a/extensions/devservices/postgresql/pom.xml b/extensions/devservices/postgresql/pom.xml
@@ -40,9 +40,9 @@
             <scope>test</scope>
         </dependency>
         <dependency>
-            <groupId>org.jboss.logmanager</groupId> 
-            <artifactId>jboss-logmanager-embedded</artifactId> 
-            <scope>test</scope> 
+            <groupId>org.jboss.logmanager</groupId>
+            <artifactId>jboss-logmanager</artifactId>
+            <scope>test</scope>
         </dependency>
     </dependencies>
     <build>