Reactive path matching does not consider servlet context path [SPR-14714]

[spring-projects-issues]

Arjen Poutsma opened SPR-14714 and commented

When mapping against path patterns, the HttpRequestPathHelper currently uses the path as returned from HttpRequest.getURI().getRawPath() as lookup path. However, when running in a servlet environment, this lookup path represents the entire path (including servlet context), which is not desirable.

Proposed solution: add a getPath() method to HttpRequest, defaulting to return getURI().getRawPath(), but with an override in ServletServerHttpRequest to return the proper lookup path (obtained from UrlPathHelper.

---

Affects: 5.0 M1

Issue Links:

• Add support for context paths in Spring Web Reactive [SPR-14726] #19291 Add support for context paths in Spring Web Reactive

[spring-projects-issues]

Arjen Poutsma commented

PR at #1174

[spring-projects-issues]

Rossen Stoyanchev commented

Resolved this by adding a getContextPath() method to ServerHttpRequest because sooner rather than later we need to consider support for that (see #19291). Either way as a short-term workaround I prefer an explicit method that we have to decide to keep or not vs a more innocent looking getPath that could get (mis)-used..