Skip to content

Update DeferredCsrfToken to implement Supplier #16892

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
wants to merge 1 commit into from
Closed

Update DeferredCsrfToken to implement Supplier #16892

wants to merge 1 commit into from

Conversation

vasanth-79
Copy link
Contributor

Issue Number: #16870

  • Modified DeferredCsrfToken interface to extend Supplier.
  • Refactored code to pass deferredCsrfToken directly instead of deferredCsrfToken::get.
  • Applied changes in both main and test folders.

@vasanth-79
Update DeferredCsrfToken to implement Supplier
29eefa1 
Issue Number: spring-projects#16870

- Modified DeferredCsrfToken interface to extend Supplier<CSRFToken>.
- Refactored code to pass deferredCsrfToken directly instead of deferredCsrfToken::get.
- Applied changes in both main and test folders.

Signed-off-by: Vasanth <[email protected]>
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Apr 6, 2025
@jzheaux jzheaux added the status: duplicate A duplicate of another issue label Apr 8, 2025
@jzheaux jzheaux self-assigned this Apr 8, 2025
@jzheaux jzheaux added in: web An issue in web modules (web, webmvc) type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged labels Apr 8, 2025
@jzheaux
Copy link
Contributor

jzheaux commented Apr 8, 2025

Thanks for the PR, @vasanth-79! I'm going to close it in favor of #16905. You can see my comment on the original ticket for details.

@jzheaux jzheaux closed this Apr 8, 2025
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) status: duplicate A duplicate of another issue type: enhancement A general enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@vasanth-79 @jzheaux @spring-projects-issues