Fix for access-log feature. (#8041)

Correct config prefix/id, and added provides for the service. Signed-off-by: Tomas Langer <[email protected]>
helidon-io · Nov 20, 2023 · aeeb6af · aeeb6af
1 parent ea3e55c
commit aeeb6af
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 2 deletions.
diff --git a/examples/todo-app/frontend/src/main/resources/application.yaml b/examples/todo-app/frontend/src/main/resources/application.yaml
@@ -18,6 +18,8 @@ env: docker
 server:
   port: 8080
   features:
+    access-log:
+      enabled: true
     security:
       paths:
         - path: "/api/{+}"

diff --git a/...ver/access-log/src/main/java/io/helidon/webserver/accesslog/AccessLogFeatureProvider.java b/...ver/access-log/src/main/java/io/helidon/webserver/accesslog/AccessLogFeatureProvider.java
@@ -21,7 +21,7 @@
 import io.helidon.webserver.spi.ServerFeatureProvider;
 
 /**
- * {@link java.util.ServiceLoader} provider implementation for context feature for {@link io.helidon.webserver.WebServer}.
+ * {@link java.util.ServiceLoader} provider implementation for access log feature for {@link io.helidon.webserver.WebServer}.
  */
 @Weight(AccessLogFeature.WEIGHT)
 public class AccessLogFeatureProvider implements ServerFeatureProvider<AccessLogFeature> {
@@ -36,7 +36,7 @@ public AccessLogFeatureProvider() {
 
     @Override
     public String configKey() {
-        return "context";
+        return "access-log";
     }
 
     @Override

diff --git a/webserver/access-log/src/main/java/module-info.java b/webserver/access-log/src/main/java/module-info.java
@@ -38,4 +38,7 @@
 
     exports io.helidon.webserver.accesslog;
 
+    provides io.helidon.webserver.spi.ServerFeatureProvider
+            with io.helidon.webserver.accesslog.AccessLogFeatureProvider;
+
 }