From d3440880c4fc70a40b4e43391a44ce74f677db3d Mon Sep 17 00:00:00 2001 From: Stephane Nicoll Date: Mon, 16 Nov 2020 07:30:08 +0100 Subject: [PATCH] Polish "Avoid unnecessary pattern compilation in WebFluxTags" See gh-24147 --- .../metrics/web/reactive/server/WebFluxTags.java | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/spring-boot-project/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/metrics/web/reactive/server/WebFluxTags.java b/spring-boot-project/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/metrics/web/reactive/server/WebFluxTags.java index 35c4a7935f..f0d4daab27 100644 --- a/spring-boot-project/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/metrics/web/reactive/server/WebFluxTags.java +++ b/spring-boot-project/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/metrics/web/reactive/server/WebFluxTags.java @@ -131,13 +131,6 @@ public final class WebFluxTags { return URI_UNKNOWN; } - private static String removeTrailingSlash(String text) { - if (!StringUtils.hasLength(text)) { - return text; - } - return text.endsWith("/") ? text.substring(0, text.length() - 1) : text; - } - private static String getPathInfo(ServerWebExchange exchange) { String path = exchange.getRequest().getPath().value(); String uri = StringUtils.hasText(path) ? path : "/"; @@ -145,6 +138,13 @@ public final class WebFluxTags { return removeTrailingSlash(singleSlashes); } + private static String removeTrailingSlash(String text) { + if (!StringUtils.hasLength(text)) { + return text; + } + return text.endsWith("/") ? text.substring(0, text.length() - 1) : text; + } + /** * Creates an {@code exception} tag based on the {@link Class#getSimpleName() simple * name} of the class of the given {@code exception}.