LTS 3.15.1 (#330)

quarkiverse · Oct 4, 2024 · 619d340 · 619d340
1 parent 534cdbc
commit 619d340
Show file tree

Hide file tree

Showing 3 changed files with 27 additions and 22 deletions.
diff --git a/deployment/src/main/java/io/quarkiverse/loggingmanager/deployment/LoggingManagerConfig.java b/deployment/src/main/java/io/quarkiverse/loggingmanager/deployment/LoggingManagerConfig.java
@@ -1,33 +1,38 @@
 package io.quarkiverse.loggingmanager.deployment;
 
-import io.quarkus.runtime.annotations.ConfigItem;
+import io.quarkus.runtime.annotations.ConfigPhase;
 import io.quarkus.runtime.annotations.ConfigRoot;
+import io.smallrye.config.ConfigMapping;
+import io.smallrye.config.WithDefault;
+import io.smallrye.config.WithName;
 
-@ConfigRoot
-public class LoggingManagerConfig {
+@ConfigMapping(prefix = "quarkus.logging-manager")
+@ConfigRoot(phase = ConfigPhase.BUILD_TIME)
+public interface LoggingManagerConfig {
 
     /**
      * The base path
      */
-    @ConfigItem(defaultValue = "logging-manager")
-    String basePath;
+    @WithDefault("logging-manager")
+    String basePath();
 
     /**
      * Whether to include the Logger Manager endpoints in the generated OpenAPI document
      */
-    @ConfigItem(name = "openapi.included", defaultValue = "false")
-    boolean openapiIncluded;
+    @WithName("openapi.included")
+    @WithDefault("false")
+    boolean openapiIncluded();
 
     /**
      * The tag to use if OpenAPI is included
      */
-    @ConfigItem(defaultValue = "Logging-manager")
-    String openapiTag;
+    @WithDefault("logging-manager")
+    String openapiTag();
 
     /**
      * Always include this. By default, this will always be included.
      * Setting this to false will also exclude this in Prod
      */
-    @ConfigItem(defaultValue = "true")
-    boolean alwaysInclude;
-}
+    @WithDefault("true")
+    boolean alwaysInclude();
+}
diff --git a/...yment/src/main/java/io/quarkiverse/loggingmanager/deployment/LoggingManagerProcessor.java b/...yment/src/main/java/io/quarkiverse/loggingmanager/deployment/LoggingManagerProcessor.java
@@ -28,7 +28,7 @@ static class OpenAPIIncluded implements BooleanSupplier {
         LoggingManagerConfig config;
 
         public boolean getAsBoolean() {
-            return config.openapiIncluded;
+            return config.openapiIncluded();
         }
     }
 
@@ -48,7 +48,7 @@ void includeRestEndpoints(BuildProducer<RouteBuildItem> routeProducer,
             LoggingManagerRuntimeConfig runtimeConfig,
             ManagementInterfaceBuildTimeConfig managementConfig) {
 
-        if ("/".equals(loggingManagerConfig.basePath)) {
+        if ("/".equals(loggingManagerConfig.basePath())) {
             throw new ConfigurationException(
                     "quarkus.logging-manager.base-path was set to \"/\", this is not allowed as it blocks the application from serving anything else.");
         }
@@ -59,15 +59,15 @@ void includeRestEndpoints(BuildProducer<RouteBuildItem> routeProducer,
 
             routeProducer.produce(nonApplicationRootPathBuildItem.routeBuilder()
                     .management()
-                    .routeFunction(loggingManagerConfig.basePath,
+                    .routeFunction(loggingManagerConfig.basePath(),
                             recorder.routeConsumer(bodyHandlerBuildItem.getHandler(), runtimeConfig))
                     .displayOnNotFoundPage("All available loggers")
                     .handler(loggerHandler)
                     .build());
 
             routeProducer.produce(nonApplicationRootPathBuildItem.routeBuilder()
                     .management()
-                    .nestedRoute(loggingManagerConfig.basePath, "levels")
+                    .nestedRoute(loggingManagerConfig.basePath(), "levels")
                     .displayOnNotFoundPage("All available log levels")
                     .handler(levelHandler)
                     .build());
@@ -86,19 +86,19 @@ public void includeInOpenAPIEndpoint(BuildProducer<AddToOpenAPIDefinitionBuildIt
         if (capabilities.isPresent(Capability.SMALLRYE_OPENAPI)
                 && shouldIncludeInOpenAPI(launchMode, loggingManagerConfig, managementConfig)) {
             LoggingManagerOpenAPIFilter filter = new LoggingManagerOpenAPIFilter(
-                    nonApplicationRootPathBuildItem.resolvePath(loggingManagerConfig.basePath),
-                    loggingManagerConfig.openapiTag);
+                    nonApplicationRootPathBuildItem.resolvePath(loggingManagerConfig.basePath()),
+                    loggingManagerConfig.openapiTag());
             openAPIProducer.produce(new AddToOpenAPIDefinitionBuildItem(filter));
         }
     }
 
     private static boolean shouldInclude(LaunchModeBuildItem launchMode, LoggingManagerConfig loggingManagerConfig) {
-        return launchMode.getLaunchMode().isDevOrTest() || loggingManagerConfig.alwaysInclude;
+        return launchMode.getLaunchMode().isDevOrTest() || loggingManagerConfig.alwaysInclude();
     }
 
     private static boolean shouldIncludeInOpenAPI(LaunchModeBuildItem launchMode, LoggingManagerConfig loggingManagerConfig,
             ManagementInterfaceBuildTimeConfig managementConfig) {
         return !managementConfig.enabled && shouldInclude(launchMode, loggingManagerConfig);
     }
 
-}
+}
diff --git a/pom.xml b/pom.xml
@@ -31,7 +31,7 @@
         <maven.compiler.release>17</maven.compiler.release>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
         <project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>
-        <quarkus.version>3.8.5</quarkus.version>
+        <quarkus.version>3.15.1</quarkus.version>
     </properties>  
     <dependencyManagement>
         <dependencies>
@@ -80,4 +80,4 @@
             </modules>
         </profile>
     </profiles>
-</project>
+</project>