Remove redundant handling of a null exchange from WebFluxTags.uri()

Closes gh-14504

Author: wilkinsona

spring-boot-project/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/metrics/web/reactive/server/WebFluxTags.java

@@ -40,8 +40,6 @@ public final class WebFluxTags {
 
 	private static final Tag URI_ROOT = Tag.of("uri", "root");
 
-	private static final Tag URI_UNKNOWN = Tag.of("uri", "UNKNOWN");
-
 	private static final Tag EXCEPTION_NONE = Tag.of("exception", "None");
 
 	private WebFluxTags() {
@@ -80,26 +78,23 @@ public static Tag status(ServerWebExchange exchange) {
 	 * @return the uri tag derived from the exchange
 	 */
 	public static Tag uri(ServerWebExchange exchange) {
-		if (exchange != null) {
-			PathPattern pathPattern = exchange
-					.getAttribute(HandlerMapping.BEST_MATCHING_PATTERN_ATTRIBUTE);
-			if (pathPattern != null) {
-				return Tag.of("uri", pathPattern.getPatternString());
-			}
-			HttpStatus status = exchange.getResponse().getStatusCode();
-			if (status != null && status.is3xxRedirection()) {
-				return URI_REDIRECTION;
-			}
-			if (status != null && status.equals(HttpStatus.NOT_FOUND)) {
-				return URI_NOT_FOUND;
-			}
-			String path = exchange.getRequest().getPath().value();
-			if (path.isEmpty()) {
-				return URI_ROOT;
-			}
-			return Tag.of("uri", path);
+		PathPattern pathPattern = exchange
+				.getAttribute(HandlerMapping.BEST_MATCHING_PATTERN_ATTRIBUTE);
+		if (pathPattern != null) {
+			return Tag.of("uri", pathPattern.getPatternString());
+		}
+		HttpStatus status = exchange.getResponse().getStatusCode();
+		if (status != null && status.is3xxRedirection()) {
+			return URI_REDIRECTION;
+		}
+		if (status != null && status.equals(HttpStatus.NOT_FOUND)) {
+			return URI_NOT_FOUND;
+		}
+		String path = exchange.getRequest().getPath().value();
+		if (path.isEmpty()) {
+			return URI_ROOT;
 		}
-		return URI_UNKNOWN;
+		return Tag.of("uri", path);
 	}
 
 	/**

spring-boot-project/spring-boot-actuator/src/test/java/org/springframework/boot/actuate/metrics/web/reactive/server/WebFluxTagsTests.java

@@ -78,12 +78,6 @@ public void uriTagToleratesCustomResponseStatus() {
 		assertThat(tag.getValue()).isEqualTo("root");
 	}
 
-	@Test
-	public void uriTagIsUnknownWhenRequestIsNull() {
-		Tag tag = WebFluxTags.uri(null);
-		assertThat(tag.getValue()).isEqualTo("UNKNOWN");
-	}
-
 	@Test
 	public void methodTagToleratesNonStandardHttpMethods() {
 		ServerWebExchange exchange = mock(ServerWebExchange.class);