Skip to content

Wrong repo#912

Closed
jkschneider wants to merge 1 commit intomainfrom
jkschneider/add-suffix-pattern-match-recipe
Closed

Wrong repo#912
jkschneider wants to merge 1 commit intomainfrom
jkschneider/add-suffix-pattern-match-recipe

Conversation

@jkschneider
Copy link
Member

@jkschneider jkschneider commented Jan 26, 2026

Wrong repo

In Spring Framework 5.2.4 and earlier, suffix pattern matching was enabled by
default. This meant @RequestMapping("/users") would match /users.json, /users.xml,
etc. Spring 5.3 deprecated this behavior and changed the default to false.

This recipe adds setUseSuffixPatternMatch(true) to WebMvcConfigurer implementations
to preserve the legacy behavior during migration. It will either add the entire
configurePathMatch() method if missing, or add the call to an existing method.

Note: This only applies to Spring MVC. WebFlux never supported suffix pattern
matching.

Fixes moderneinc/customer-requests#1305
@github-project-automation github-project-automation bot moved this to In Progress in OpenRewrite Jan 26, 2026
@jkschneider jkschneider deleted the jkschneider/add-suffix-pattern-match-recipe branch January 26, 2026 15:29
@github-project-automation github-project-automation bot moved this from In Progress to Done in OpenRewrite Jan 26, 2026
@jkschneider jkschneider changed the title Add AddSetUseSuffixPatternMatch recipe for Spring 5.3 migration Wrong repo Jan 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

1 participant