Rest client per CDI injected client logging configuration

quarkusio · Jan 11, 2025 · 1d1c6be · 1d1c6be
1 parent 4b0ff98
commit 1d1c6be
Show file tree

Hide file tree

Showing 5 changed files with 30 additions and 1 deletion.
diff --git a/docs/src/main/asciidoc/rest-client.adoc b/docs/src/main/asciidoc/rest-client.adoc
@@ -1845,14 +1845,20 @@ As HTTP messages can have large bodies, we limit the amount of body characters l
 
 NOTE: REST Client is logging the traffic with level DEBUG and does not alter logger properties. You may need to adjust your logger configuration to use this feature.
 
+These configuration properties work globally for all clients injected by CDI.
+If you want configure logging for particular one (but only for CDI injected), you should do it by specifying named "client" properties, also known as `quarkus.rest-client."client".logging.*` properties.
+
 An example logging configuration:
 
 [source,properties]
 ----
 quarkus.rest-client.logging.scope=request-response
 quarkus.rest-client.logging.body-limit=50
 
+quarkus.rest-client.extensions-api.scope=all
+
 quarkus.log.category."org.jboss.resteasy.reactive.client.logging".level=DEBUG
+quarkus.log.console.level=DEBUG
 ----
 
 [TIP]

diff --git a/...t-client-config/runtime/src/main/java/io/quarkus/restclient/config/RestClientsConfig.java b/...t-client-config/runtime/src/main/java/io/quarkus/restclient/config/RestClientsConfig.java
@@ -621,6 +621,11 @@ default Optional<String> uriReload() {
          */
         @WithDefault("${microprofile.rest.client.disable.default.mapper:false}")
         Boolean disableDefaultMapper();
+
+        /**
+         * Logging configuration.
+         */
+        Optional<RestClientLoggingConfig> logging();
     }
 
     class RestClientKeysProvider implements Supplier<Iterable<String>> {

diff --git a/.../runtime/src/main/java/io/quarkus/rest/client/reactive/runtime/RestClientBuilderImpl.java b/.../runtime/src/main/java/io/quarkus/rest/client/reactive/runtime/RestClientBuilderImpl.java
@@ -440,7 +440,10 @@ public <T> T build(Class<T> aClass) throws IllegalStateException, RestClientDefi
         clientBuilder.register(new MicroProfileRestClientResponseFilter(exceptionMappers));
         clientBuilder.followRedirects(followRedirects);
 
-        RestClientsConfig.RestClientLoggingConfig logging = restClients.logging();
+        RestClientsConfig.RestClientLoggingConfig logging = getConfiguration().hasProperty(QuarkusRestClientProperties.LOGGING)
+                ? (RestClientsConfig.RestClientLoggingConfig) getConfiguration()
+                        .getProperty(QuarkusRestClientProperties.LOGGING)
+                : restClients.logging();
 
         LoggingScope effectiveLoggingScope = loggingScope; // if a scope was specified programmatically, it takes precedence
         if (effectiveLoggingScope == null) {

diff --git a/...e/src/main/java/io/quarkus/rest/client/reactive/runtime/RestClientCDIDelegateBuilder.java b/...e/src/main/java/io/quarkus/rest/client/reactive/runtime/RestClientCDIDelegateBuilder.java
@@ -81,9 +81,19 @@ void configureBuilder(QuarkusRestClientBuilder builder) {
         configureQueryParamStyle(builder);
         configureProxy(builder);
         configureShared(builder);
+        configureLogging(builder);
         configureCustomProperties(builder);
     }
 
+    private void configureLogging(QuarkusRestClientBuilder builder) {
+        Optional<RestClientsConfig.RestClientLoggingConfig> logging = restClientConfig.logging();
+        if (logging.isPresent()) {
+            builder.property(QuarkusRestClientProperties.LOGGING, logging.get());
+        } else if (configRoot.logging() != null) {
+            builder.property(QuarkusRestClientProperties.LOGGING, configRoot.logging());
+        }
+    }
+
     private void configureCustomProperties(QuarkusRestClientBuilder builder) {
         Optional<String> encoder = oneOf(restClientConfig.multipartPostEncoderMode(), configRoot.multipartPostEncoderMode());
         if (encoder != null && encoder.isPresent()) {

diff --git a/...ime/src/main/java/org/jboss/resteasy/reactive/client/api/QuarkusRestClientProperties.java b/...ime/src/main/java/org/jboss/resteasy/reactive/client/api/QuarkusRestClientProperties.java
@@ -85,4 +85,9 @@ public class QuarkusRestClientProperties {
      */
     public static final String CAPTURE_STACKTRACE = "io.quarkus.rest.client.capture-stacktrace";
 
+    /**
+     * logging configuration
+     */
+    public static final String LOGGING = "io.quarkus.rest.client.logging";
+
 }