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

Perform checks for bean validation constraints in HandlerMethod only when needed #32007

Closed
rstoyanchev opened this issue Jan 10, 2024 · 1 comment
Closed

Perform checks for bean validation constraints in HandlerMethod only when needed #32007

rstoyanchev opened this issue Jan 10, 2024 · 1 comment
Assignees
@rstoyanchev
Labels
in: web Issues in web modules (web, webmvc, webflux, websocket) type: enhancement A general enhancement
Milestone
6.1.3

Comments

@rstoyanchev
Copy link
Contributor

Method validation is not needed for all types of controller methods. We should make the initialization of the shouldValidate flags in HandlerMethod more explicit rather than performed for every instances created.
@rstoyanchev rstoyanchev added in: web Issues in web modules (web, webmvc, webflux, websocket) type: enhancement A general enhancement labels Jan 10, 2024
@rstoyanchev rstoyanchev added this to the 6.1.3 milestone Jan 10, 2024
@rstoyanchev rstoyanchev self-assigned this Jan 10, 2024
@mauriciogeneroso mauriciogeneroso mentioned this issue Feb 1, 2024
Closed
@tadlakha9
Copy link

tadlakha9 commented Mar 7, 2024 •
edited
Loading

Hello @rstoyanchev, Do we have to enable any specific flag to enable the method validation? As in our project we are getting issue while validation is not passed

@notizklotz notizklotz mentioned this issue Mar 8, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rstoyanchev rstoyanchev

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

No branches or pull requests
2 participants
@rstoyanchev @tadlakha9