From f67caaf82c9c339fafa9191152c4185912565e64 Mon Sep 17 00:00:00 2001 From: Georgios Andrianakis Date: Tue, 5 Sep 2023 14:05:55 +0300 Subject: [PATCH] Fix invalid logging pattern Closes: #35614 (cherry picked from commit f2a2516f4df0e2e76f6bab3d40895b5a3fcf093f) --- .../vertx/OpenTelemetryVertxTracingFactory.java | 8 ++++---- .../deployment/MicroProfileRestClientEnricher.java | 2 +- .../reactive/deployment/RestClientReactiveProcessor.java | 2 +- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/extensions/opentelemetry/runtime/src/main/java/io/quarkus/opentelemetry/runtime/tracing/intrumentation/vertx/OpenTelemetryVertxTracingFactory.java b/extensions/opentelemetry/runtime/src/main/java/io/quarkus/opentelemetry/runtime/tracing/intrumentation/vertx/OpenTelemetryVertxTracingFactory.java index 56060eaa5d16f..31e4992288014 100644 --- a/extensions/opentelemetry/runtime/src/main/java/io/quarkus/opentelemetry/runtime/tracing/intrumentation/vertx/OpenTelemetryVertxTracingFactory.java +++ b/extensions/opentelemetry/runtime/src/main/java/io/quarkus/opentelemetry/runtime/tracing/intrumentation/vertx/OpenTelemetryVertxTracingFactory.java @@ -52,7 +52,7 @@ public Object receiveRequest( final TagExtractor tagExtractor) { if (delegate == null) { log.warnv("VertxTracer delegate not set. Will not submit this trace. " + - "SpanKind: {}; Request: {}; Operation:{}.", + "SpanKind: {0}; Request: {1}; Operation:{2}.", kind, request == null ? "null" : request.toString(), operation); @@ -70,7 +70,7 @@ public void sendResponse( final TagExtractor tagExtractor) { if (delegate == null) { log.warnv("VertxTracer delegate not set. Will not submit this trace. " + - "Response: {}; Failure: {}.", + "Response: {0}; Failure: {1}.", response == null ? "null" : response.toString(), failure == null ? "null" : failure.getMessage()); return; @@ -89,7 +89,7 @@ public Object sendRequest( final TagExtractor tagExtractor) { if (delegate == null) { log.warnv("VertxTracer delegate not set. Will not submit this trace. " + - "SpanKind: {}; Request: {}; Operation:{}.", + "SpanKind: {0}; Request: {1}; Operation:{2}.", kind, request == null ? "null" : request.toString(), operation); @@ -107,7 +107,7 @@ public void receiveResponse( final TagExtractor tagExtractor) { if (delegate == null) { log.warnv("VertxTracer delegate not set. Will not submit this trace. " + - "Response: {}; Failure: {}.", + "Response: {0}; Failure: {1}.", response == null ? "null" : response.toString(), failure == null ? "null" : failure.getMessage()); return; diff --git a/extensions/resteasy-reactive/rest-client-reactive/deployment/src/main/java/io/quarkus/rest/client/reactive/deployment/MicroProfileRestClientEnricher.java b/extensions/resteasy-reactive/rest-client-reactive/deployment/src/main/java/io/quarkus/rest/client/reactive/deployment/MicroProfileRestClientEnricher.java index 5f22e57c186b3..ef40abfd07b79 100644 --- a/extensions/resteasy-reactive/rest-client-reactive/deployment/src/main/java/io/quarkus/rest/client/reactive/deployment/MicroProfileRestClientEnricher.java +++ b/extensions/resteasy-reactive/rest-client-reactive/deployment/src/main/java/io/quarkus/rest/client/reactive/deployment/MicroProfileRestClientEnricher.java @@ -540,7 +540,7 @@ private void addHeaderParam(MethodInfo declaringMethod, MethodCreator fillHeader String[] values = annotation.value().asStringArray(); if (values.length == 0) { - log.warnv("Ignoring ClientHeaderParam that specifies an empty array of header values for header {} on {}", + log.warnv("Ignoring ClientHeaderParam that specifies an empty array of header values for header {0} on {1}", annotation.value("name").asString(), annotation.target()); return; } diff --git a/extensions/resteasy-reactive/rest-client-reactive/deployment/src/main/java/io/quarkus/rest/client/reactive/deployment/RestClientReactiveProcessor.java b/extensions/resteasy-reactive/rest-client-reactive/deployment/src/main/java/io/quarkus/rest/client/reactive/deployment/RestClientReactiveProcessor.java index b4fba326dc0fc..066d733223757 100644 --- a/extensions/resteasy-reactive/rest-client-reactive/deployment/src/main/java/io/quarkus/rest/client/reactive/deployment/RestClientReactiveProcessor.java +++ b/extensions/resteasy-reactive/rest-client-reactive/deployment/src/main/java/io/quarkus/rest/client/reactive/deployment/RestClientReactiveProcessor.java @@ -693,7 +693,7 @@ private ScopeInfo computeDefaultScope(Capabilities capabilities, Config config, BuiltinScope globalDefaultScope = BuiltinScope.from(DotName.createSimple(mpClientConfig.scope)); if (globalDefaultScope == null) { - log.warnv("Unable to map the global rest client scope: '{}' to a scope. Using @ApplicationScoped", + log.warnv("Unable to map the global rest client scope: '{0}' to a scope. Using @ApplicationScoped", mpClientConfig.scope); globalDefaultScope = BuiltinScope.APPLICATION; }