Update requestRejectedHandler via setter

[Drezir]

Ability to set requestRejectedHandler via public settter in method WebSecurityConfigurerAdapter.configure(WebSecurity)

[pivotal-issuemaster]

@Drezir Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

[pivotal-issuemaster]

@Drezir Thank you for signing the Contributor License Agreement!

[rwinch]

Thanks for the PR @Drezir! For my understanding...Can you explain why you are wanting to provide a method on the DSL rather than picking up the RequestRejectedHandler as a Bean?

[Drezir]

I thought it should be there for consistency. I think that other WebMvcCinfigurerAdapter entities are also registrable using beans or methods.

[rwinch]

Thanks for the explanation. Initially my thought was that there is only a need for one RequestRejectedHandler, but perhaps users are using a composite approach. Consistency is another good reason. Would you be able to add tests and fix the checkstyle/formatting error? To see it run ./gradlew :spring-security-web:check https://github.com/spring-projects/spring-security/pull/8985/checks?check_run_id=1037486294

[Drezir]

Thanks for the explanation. Initially my thought was that there is only a need for one RequestRejectedHandler, but perhaps users are using a composite approach. Consistency is another good reason. Would you be able to add tests and fix the checkstyle/formatting error? To see it run ./gradlew :spring-security-web:check https://github.com/spring-projects/spring-security/pull/8985/checks?check_run_id=1037486294

Sure, I will try it.

[dawi]

This would be a nice addition.
Then I would be able to configure different RequestRejectedHandlers for Webapp and Webservices.

• For / I would redirect the user to an error page.
• For /api I would render a meaningful json response.

Currently I have to implement one RequestRejectedHandler, which in our case is not nice,
because we have divided Webapp and Webservices into different modules.

[jzheaux]

@Drezir Are you able to apply the requested changes?

[jgrandja]

Closing because the requested changes have not been made. Feel free to create a new PR with the requested changes.