Skip to content

Allow separator configuration in PathPatternParser #23092

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
Closed
bclozel opened this issue Jun 6, 2019 · 0 comments
Closed

Allow separator configuration in PathPatternParser #23092

bclozel opened this issue Jun 6, 2019 · 0 comments
Assignees
@bclozel
Labels
in: web Issues in web modules (web, webmvc, webflux, websocket) type: enhancement A general enhancement
Milestone
5.2 M3

Comments

@bclozel
Copy link
Member

bclozel commented Jun 6, 2019

Since #22642, the new RouteMatcher interface allows to choose various implementation for matching routes in the MVC, WebFlux and Messaging infrastructures.

This issue is a follow up of #22642 and prepares PathPatternParser for spring-projects/spring-boot#16865: in order to use PathPatternParser in a messaging context, we need to change the default path separator "/" to something appropriate for messaging paths, usually the "." separator.

This issue should provide a way to configure the path separator with PathPatternParser.
@bclozel bclozel added in: web Issues in web modules (web, webmvc, webflux, websocket) type: enhancement A general enhancement labels Jun 6, 2019
@bclozel bclozel added this to the 5.2 M3 milestone Jun 6, 2019
@bclozel bclozel self-assigned this Jun 6, 2019
@bclozel bclozel closed this as completed in 33becd8 Jun 6, 2019
This was referenced Jun 20, 2019
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bclozel bclozel

Labels
in: web Issues in web modules (web, webmvc, webflux, websocket) type: enhancement A general enhancement
Projects
None yet
Milestone
5.2 M3
Development

No branches or pull requests
1 participant
@bclozel