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

Remove LazyCsrfTokenRepository usage #13202

Merged
merged 1 commit into from Jun 22, 2023
Merged

Remove LazyCsrfTokenRepository usage #13202

merged 1 commit into from Jun 22, 2023

Conversation

kandaguru17
Copy link
Contributor

@kandaguru17 kandaguru17 commented May 21, 2023 •
edited

Remove usage of deprecated LazyCsrfTokenRepository from CsrfConfigurer.java and CsrfBeanDefinitionParser.java
as CsrfFilter uses CsrfTokenRepository.

Closes gh-13194

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label May 21, 2023
@kandaguru17 kandaguru17 force-pushed the gh-13194 branch 4 times, most recently from eaa55db to 5eca60d Compare May 21, 2023 09:24
@jzheaux jzheaux added in: web An issue in web modules (web, webmvc) type: enhancement A general enhancement type: breaks-passivity A change that breaks passivity with the previous release and removed status: waiting-for-triage An issue we've not yet triaged labels May 22, 2023
@jzheaux jzheaux self-assigned this May 22, 2023
@jzheaux jzheaux removed the type: breaks-passivity A change that breaks passivity with the previous release label May 22, 2023
@kandaguru17 kandaguru17 marked this pull request as ready for review May 22, 2023 23:50
@kandaguru17 kandaguru17 marked this pull request as draft May 23, 2023 00:21
@kandaguru17 kandaguru17 marked this pull request as ready for review May 23, 2023 00:29
@jzheaux jzheaux added this to the 6.2.0-M1 milestone May 24, 2023
@jzheaux
Copy link
Contributor

jzheaux commented May 24, 2023

Thanks for the PR, @kandaguru17. Once the 6.1.x branch is created, we can merge this to main.

@jzheaux jzheaux merged commit 7e01ebd into spring-projects:main Jun 22, 2023
2 checks passed
@jzheaux
Copy link
Contributor

jzheaux commented Jun 22, 2023

Thanks again @kandaguru17! This is now merged into main.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@jzheaux jzheaux

Labels
in: web An issue in web modules (web, webmvc) type: enhancement A general enhancement
Projects
None yet
Milestone
6.2.0-M1
Development

Successfully merging this pull request may close these issues.

Remove LazyCsrfTokenRepository usage
3 participants
@kandaguru17 @jzheaux @spring-projects-issues