Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Trailing slash required with Undertow + custom root path #8475

Closed
dteleguin opened this issue Apr 8, 2020 · 1 comment · Fixed by #8608
Closed

Trailing slash required with Undertow + custom root path #8475

dteleguin opened this issue Apr 8, 2020 · 1 comment · Fixed by #8608
Labels
kind/bug Something isn't working
Milestone

Comments

@dteleguin
Copy link
Contributor

Describe the bug
When using quarkus-undertow in combination with custom root path, a trailing slash has to be appended to the app path. This is not required when using vert.x. Additionally, Java™ Servlet Specification assumes that the slash shouldn't be required:

A ServletContext is rooted at a known path within a Web server. For example, a
servlet context could be located at http://www.mycorp.com/catalog. All requests
that begin with the /catalog request path, known as the context path, are routed to
the Web application associated with the ServletContext.

Expected behavior
GET http://localhost:8080/foo => 200 OK (Welcome page or root resource)
GET http://localhost:8080/foo/ => 200 OK (ditto)

Actual behavior
GET http://localhost:8080/foo => 404 Not Found + "Resource not found" page
GET http://localhost:8080/foo/ => 200 OK

Configuration

quarkus.servlet.context-path = /foo

Environment (please complete the following information):

  • Quarkus version or git rev: 1.3.1.Final
@dteleguin dteleguin added the kind/bug Something isn't working label Apr 8, 2020
@sberyozkin
Copy link
Member

Hi Stuart @stuartwdouglas is it somewhere in Quarkus or in Undertow itself where it is handled ?

stuartwdouglas added a commit to stuartwdouglas/quarkus that referenced this issue Apr 16, 2020
stuartwdouglas added a commit to stuartwdouglas/quarkus that referenced this issue Apr 16, 2020
@gsmet gsmet added this to the 1.4.0.Final milestone Apr 20, 2020
gsmet pushed a commit to gsmet/quarkus that referenced this issue Apr 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants