diff --git a/plugins/spring/src/main/java/com/navercorp/pinpoint/plugin/spring/web/jakarta/interceptor/ExposePathWithinMappingInterceptor.java b/plugins/spring/src/main/java/com/navercorp/pinpoint/plugin/spring/web/jakarta/interceptor/ExposePathWithinMappingInterceptor.java index e13312b6f64d..98569e0a1c0f 100644 --- a/plugins/spring/src/main/java/com/navercorp/pinpoint/plugin/spring/web/jakarta/interceptor/ExposePathWithinMappingInterceptor.java +++ b/plugins/spring/src/main/java/com/navercorp/pinpoint/plugin/spring/web/jakarta/interceptor/ExposePathWithinMappingInterceptor.java @@ -13,6 +13,7 @@ public class ExposePathWithinMappingInterceptor implements AroundInterceptor { private final PLogger logger = PLoggerFactory.getLogger(getClass()); + private final boolean isDebug = logger.isDebugEnabled(); private final TraceContext traceContext; private final Boolean uriStatCollectMethod; @@ -35,7 +36,11 @@ public void after(Object target, Object[] args, Object result, Throwable throwab try { final SpanRecorder spanRecorder = trace.getSpanRecorder(); final String url = ArrayArgumentUtils.getArgument(args, 0, String.class); - logger.debug("Attempt recording URI with template: {}", url); + + if (isDebug) { + logger.debug("Attempt recording URI with template: {}", url); + } + if (StringUtils.hasLength(url)) { spanRecorder.recordUriTemplate(url); } diff --git a/plugins/spring/src/main/java/com/navercorp/pinpoint/plugin/spring/web/jakarta/interceptor/LookupHandlerMethodInterceptor.java b/plugins/spring/src/main/java/com/navercorp/pinpoint/plugin/spring/web/jakarta/interceptor/LookupHandlerMethodInterceptor.java index b28cde724a54..8bce00b25b1d 100644 --- a/plugins/spring/src/main/java/com/navercorp/pinpoint/plugin/spring/web/jakarta/interceptor/LookupHandlerMethodInterceptor.java +++ b/plugins/spring/src/main/java/com/navercorp/pinpoint/plugin/spring/web/jakarta/interceptor/LookupHandlerMethodInterceptor.java @@ -16,6 +16,7 @@ public class LookupHandlerMethodInterceptor implements AroundInterceptor { private final PLogger logger = PLoggerFactory.getLogger(getClass()); + private final boolean isDebug = logger.isDebugEnabled(); private final TraceContext traceContext; private final Boolean uriStatCollectMethod; @@ -41,7 +42,11 @@ public void after(Object target, Object[] args, Object result, Throwable throwab final SpanRecorder spanRecorder = trace.getSpanRecorder(); final String uri = ServletRequestAttributeUtils.extractAttribute(request, SpringWebMvcConstants.SPRING_MVC_DEFAULT_URI_ATTRIBUTE_KEYS); - logger.debug("Attempt recording URI with template: {}", uri); + + if (isDebug) { + logger.debug("Attempt recording URI with template: {}", uri); + } + if (StringUtils.hasLength(uri)) { spanRecorder.recordUriTemplate(uri); } diff --git a/plugins/spring/src/main/java/com/navercorp/pinpoint/plugin/spring/web/jakarta/interceptor/ProcessRequestInterceptor.java b/plugins/spring/src/main/java/com/navercorp/pinpoint/plugin/spring/web/jakarta/interceptor/ProcessRequestInterceptor.java index e2f1fcb50325..4ec589f2e787 100644 --- a/plugins/spring/src/main/java/com/navercorp/pinpoint/plugin/spring/web/jakarta/interceptor/ProcessRequestInterceptor.java +++ b/plugins/spring/src/main/java/com/navercorp/pinpoint/plugin/spring/web/jakarta/interceptor/ProcessRequestInterceptor.java @@ -14,6 +14,7 @@ public class ProcessRequestInterceptor implements AroundInterceptor { private final PLogger logger = PLoggerFactory.getLogger(getClass()); + private final boolean isDebug = logger.isDebugEnabled(); private final TraceContext traceContext; private final Boolean uriStatUseUserInput; @@ -41,7 +42,11 @@ public void after(Object target, Object[] args, Object result, Throwable throwab final ServletRequest request = ArrayArgumentUtils.getArgument(args, 0, ServletRequest.class); if (request != null) { final String uri = ServletRequestAttributeUtils.extractAttribute(request, SpringWebMvcConstants.SPRING_MVC_URI_USER_INPUT_ATTRIBUTE_KEYS); - logger.debug("Attempt recording URI with template: {}", uri); + + if (isDebug) { + logger.debug("Attempt recording URI with template: {}", uri); + } + if (StringUtils.hasLength(uri)) { final SpanRecorder spanRecorder = trace.getSpanRecorder(); spanRecorder.recordUriTemplate(uri, true); diff --git a/plugins/spring/src/main/java/com/navercorp/pinpoint/plugin/spring/web/javax/interceptor/ExposePathWithinMappingInterceptor.java b/plugins/spring/src/main/java/com/navercorp/pinpoint/plugin/spring/web/javax/interceptor/ExposePathWithinMappingInterceptor.java index 2992faf7a36f..d0166c84ee57 100644 --- a/plugins/spring/src/main/java/com/navercorp/pinpoint/plugin/spring/web/javax/interceptor/ExposePathWithinMappingInterceptor.java +++ b/plugins/spring/src/main/java/com/navercorp/pinpoint/plugin/spring/web/javax/interceptor/ExposePathWithinMappingInterceptor.java @@ -12,6 +12,7 @@ public class ExposePathWithinMappingInterceptor implements AroundInterceptor { private final PLogger logger = PLoggerFactory.getLogger(getClass()); + private final boolean isDebug = logger.isDebugEnabled(); private final TraceContext traceContext; private final Boolean uriStatCollectMethod; @@ -34,7 +35,11 @@ public void after(Object target, Object[] args, Object result, Throwable throwab try { final SpanRecorder spanRecorder = trace.getSpanRecorder(); final String url = ArrayArgumentUtils.getArgument(args, 0, String.class); - logger.debug("Attempt recording URI with template: {}", url); + + if (isDebug) { + logger.debug("Attempt recording URI with template: {}", url); + } + if (StringUtils.hasLength(url)) { spanRecorder.recordUriTemplate(url); } diff --git a/plugins/spring/src/main/java/com/navercorp/pinpoint/plugin/spring/web/javax/interceptor/LookupHandlerMethodInterceptor.java b/plugins/spring/src/main/java/com/navercorp/pinpoint/plugin/spring/web/javax/interceptor/LookupHandlerMethodInterceptor.java index cc6386299f20..bc3ef2b808fd 100644 --- a/plugins/spring/src/main/java/com/navercorp/pinpoint/plugin/spring/web/javax/interceptor/LookupHandlerMethodInterceptor.java +++ b/plugins/spring/src/main/java/com/navercorp/pinpoint/plugin/spring/web/javax/interceptor/LookupHandlerMethodInterceptor.java @@ -15,6 +15,7 @@ public class LookupHandlerMethodInterceptor implements AroundInterceptor { private final PLogger logger = PLoggerFactory.getLogger(getClass()); + private final boolean isDebug = logger.isDebugEnabled(); private final TraceContext traceContext; private final Boolean uriStatCollectMethod; @@ -40,7 +41,11 @@ public void after(Object target, Object[] args, Object result, Throwable throwab final SpanRecorder spanRecorder = trace.getSpanRecorder(); final String uri = ServletRequestAttributeUtils.extractAttribute(request, SpringWebMvcConstants.SPRING_MVC_DEFAULT_URI_ATTRIBUTE_KEYS); - logger.debug("Attempt recording URI with template: {}", uri); + + if (isDebug) { + logger.debug("Attempt recording URI with template: {}", uri); + } + if (StringUtils.hasLength(uri)) { spanRecorder.recordUriTemplate(uri); } diff --git a/plugins/spring/src/main/java/com/navercorp/pinpoint/plugin/spring/web/javax/interceptor/ProcessRequestInterceptor.java b/plugins/spring/src/main/java/com/navercorp/pinpoint/plugin/spring/web/javax/interceptor/ProcessRequestInterceptor.java index 7cc66d33f14a..bd57e51b06eb 100644 --- a/plugins/spring/src/main/java/com/navercorp/pinpoint/plugin/spring/web/javax/interceptor/ProcessRequestInterceptor.java +++ b/plugins/spring/src/main/java/com/navercorp/pinpoint/plugin/spring/web/javax/interceptor/ProcessRequestInterceptor.java @@ -14,6 +14,7 @@ public class ProcessRequestInterceptor implements AroundInterceptor { private final PLogger logger = PLoggerFactory.getLogger(getClass()); + private final boolean isDebug = logger.isDebugEnabled(); private final TraceContext traceContext; private final Boolean uriStatUseUserInput; @@ -41,7 +42,11 @@ public void after(Object target, Object[] args, Object result, Throwable throwab final ServletRequest request = ArrayArgumentUtils.getArgument(args, 0, ServletRequest.class); if (request != null) { final String uri = ServletRequestAttributeUtils.extractAttribute(request, SpringWebMvcConstants.SPRING_MVC_URI_USER_INPUT_ATTRIBUTE_KEYS); - logger.debug("Attempt recording URI with template: {}", uri); + + if (isDebug) { + logger.debug("Attempt recording URI with template: {}", uri); + } + if (StringUtils.hasLength(uri)) { final SpanRecorder spanRecorder = trace.getSpanRecorder(); spanRecorder.recordUriTemplate(uri, true); diff --git a/plugins/tomcat/src/main/java/com/navercorp/pinpoint/plugin/tomcat/jakarta/interceptor/StandardHostValveInvokeInterceptor.java b/plugins/tomcat/src/main/java/com/navercorp/pinpoint/plugin/tomcat/jakarta/interceptor/StandardHostValveInvokeInterceptor.java index 3a358b05ebe2..e7c04ac946a5 100644 --- a/plugins/tomcat/src/main/java/com/navercorp/pinpoint/plugin/tomcat/jakarta/interceptor/StandardHostValveInvokeInterceptor.java +++ b/plugins/tomcat/src/main/java/com/navercorp/pinpoint/plugin/tomcat/jakarta/interceptor/StandardHostValveInvokeInterceptor.java @@ -157,7 +157,10 @@ public void after(Object target, Object[] args, Object result, Throwable throwab } } } - logger.debug("Raw Request URI: {}", request.getRequestURI()); + + if (isDebug) { + logger.debug("Raw Request URI: {}", request.getRequestURI()); + } if (uriStatEnable && uriStatUseUserInput) { recordUserUriTemplate(request); } diff --git a/plugins/tomcat/src/main/java/com/navercorp/pinpoint/plugin/tomcat/javax/interceptor/StandardHostValveInvokeInterceptor.java b/plugins/tomcat/src/main/java/com/navercorp/pinpoint/plugin/tomcat/javax/interceptor/StandardHostValveInvokeInterceptor.java index 41afd8347d7e..02aa2d802582 100644 --- a/plugins/tomcat/src/main/java/com/navercorp/pinpoint/plugin/tomcat/javax/interceptor/StandardHostValveInvokeInterceptor.java +++ b/plugins/tomcat/src/main/java/com/navercorp/pinpoint/plugin/tomcat/javax/interceptor/StandardHostValveInvokeInterceptor.java @@ -159,7 +159,9 @@ public void after(Object target, Object[] args, Object result, Throwable throwab } } } - logger.debug("Raw Request URI: {}", request.getRequestURI()); + if (isDebug) { + logger.debug("Raw Request URI: {}", request.getRequestURI()); + } if (uriStatEnable && uriStatUseUserInput) { recordUserUriTemplate(request); }