ServletContext getContextPath() does not end with forward slash.

[pmd1nh]

ServletContext.getContextPath() - The context path always comes first in a request URI. The path starts with a "/" character but does not end with a "/" character.

Currently getContextPath() does not check for the ending slash and includes it in the return.
Fix: check and remove the ending slash. An info will be logged when this happens. The flow continues.

Workaround: remove the configured ending / in the context root.

[pmd1nh]

#14400

[pmd1nh]

Additional information:

https://jakarta.ee/specifications/platform/8/apidocs/javax/servlet/ServletContext.html#getContextPath--

=====
String getContextPath()

Returns the context path of the web application.

The context path is the portion of the request URI that is used to select the context of the request. The context path always comes first in a request URI. If this context is the "root" context rooted at the base of the Web server's URL name space, this path will be an empty string. Otherwise, if the context is not rooted at the root of the server's name space, the path starts with a / character but does not end with a / character.

It is possible that a servlet container may match a context by more than one context path. In such cases the HttpServletRequest.getContextPath() will return the actual context path used by the request and it may differ from the path returned by this method. The context path returned by this method should be considered as the prime or preferred context path of the application.

=====

[mbroz2]

The vote for "Do we Agree to change the default behavior of future versions of Java Servlet (starting with 5.0) to strip the trailing / (forward slash) if present" has been POC Approved.

The vote for whether we want to change the default behavior for the existing versions of Java Servlet has also been POC Approved

[bmarwell]

@mbroz2 @pnicolucci can this be merged at some point? :)

[pnicolucci]

@bmarwell yes we're working to get this delivered in the next couple of days! Sorry for the delay!

[pmd1nh]

merged.