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.

Sign up for GitHub

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

Jump to bottom

Allow empty path in MockMvc #28823

Closed
rstoyanchev opened this issue Jul 14, 2022 · 0 comments
Closed

Allow empty path in MockMvc #28823

rstoyanchev opened this issue Jul 14, 2022 · 0 comments
Assignees
@rstoyanchev
Labels
in: test Issues in the test module in: web Issues in web modules (web, webmvc, webflux, websocket) type: enhancement A general enhancement
Milestone
6.0.0-M6

Comments

@rstoyanchev
Copy link
Contributor

In #24556 the use of a path without a leading slash was disallowed to avoid ambiguity between a path without a leading slash and a URL with a host but without a scheme. We could however allow an empty path which does not have the same issue.

This has come up after #28552, which turned off trailing slash support by default, and that in turn means that by default empty mappings only match to an empty input path.
@rstoyanchev rstoyanchev added in: test Issues in the test module in: web Issues in web modules (web, webmvc, webflux, websocket) type: enhancement A general enhancement labels Jul 14, 2022
@rstoyanchev rstoyanchev added this to the 6.0.0-M6 milestone Jul 14, 2022
@rstoyanchev rstoyanchev self-assigned this Jul 14, 2022
@rstoyanchev rstoyanchev mentioned this issue Feb 14, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rstoyanchev rstoyanchev

Labels
in: test Issues in the test module in: web Issues in web modules (web, webmvc, webflux, websocket) type: enhancement A general enhancement
Projects
None yet
Milestone
6.0.0-M6
Development

No branches or pull requests
1 participant
@rstoyanchev