diff --git a/spring-web/src/main/java/org/springframework/http/server/reactive/DefaultPathSegmentContainer.java b/spring-web/src/main/java/org/springframework/http/server/reactive/DefaultPathSegmentContainer.java index f717bba05a..b8c5c8400c 100644 --- a/spring-web/src/main/java/org/springframework/http/server/reactive/DefaultPathSegmentContainer.java +++ b/spring-web/src/main/java/org/springframework/http/server/reactive/DefaultPathSegmentContainer.java @@ -190,13 +190,13 @@ class DefaultPathSegmentContainer implements PathSegmentContainer { } static PathSegmentContainer subPath(PathSegmentContainer container, int fromIndex, int toIndex) { - if (fromIndex == toIndex) { - return EMPTY_PATH; - } List segments = container.pathSegments(); if (fromIndex == 0 && toIndex == segments.size()) { return container; } + if (fromIndex == toIndex) { + return EMPTY_PATH; + } Assert.isTrue(fromIndex < toIndex, "fromIndex: " + fromIndex + " should be < toIndex " + toIndex); Assert.isTrue(fromIndex >= 0 && fromIndex < segments.size(), "Invalid fromIndex: " + fromIndex); diff --git a/spring-web/src/test/java/org/springframework/http/server/reactive/DefaultPathSegmentContainerTests.java b/spring-web/src/test/java/org/springframework/http/server/reactive/DefaultPathSegmentContainerTests.java index 9c56be155e..836319e3b8 100644 --- a/spring-web/src/test/java/org/springframework/http/server/reactive/DefaultPathSegmentContainerTests.java +++ b/spring-web/src/test/java/org/springframework/http/server/reactive/DefaultPathSegmentContainerTests.java @@ -130,6 +130,10 @@ public class DefaultPathSegmentContainerTests { assertEquals("/b/c", PathSegmentContainer.subPath(path, 1).value()); assertEquals("/c", PathSegmentContainer.subPath(path, 2).value()); + // root path + path = PathSegmentContainer.parse("/", UTF_8); + assertEquals("/", PathSegmentContainer.subPath(path, 0).value()); + // trailing slash path = PathSegmentContainer.parse("/a/b/", UTF_8); assertEquals("/b/", PathSegmentContainer.subPath(path, 1).value()); diff --git a/spring-web/src/test/java/org/springframework/http/server/reactive/DefaultRequestPathTests.java b/spring-web/src/test/java/org/springframework/http/server/reactive/DefaultRequestPathTests.java index 1e9219bc19..20d126bd24 100644 --- a/spring-web/src/test/java/org/springframework/http/server/reactive/DefaultRequestPathTests.java +++ b/spring-web/src/test/java/org/springframework/http/server/reactive/DefaultRequestPathTests.java @@ -39,6 +39,9 @@ public class DefaultRequestPathTests { // context path only testRequestPath("/a/b", "/a/b", "", false, true, false); + // root path + testRequestPath("/", "", "/", false, true, false); + // empty path testRequestPath("", "", "", true, false, false); testRequestPath("", "/", "", true, false, false);