Skip to content

SecurityContextRequestPostProcessorSupport small-improvements #6858

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.

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

Closed
wants to merge 2 commits into from
Closed

Conversation

ch4mpy
Copy link
Contributor

@ch4mpy ch4mpy commented May 11, 2019

fixes #6857

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label May 11, 2019
@rwinch rwinch added in: test An issue in spring-security-test status: waiting-for-feedback We need additional information before we can continue and removed status: waiting-for-triage An issue we've not yet triaged labels May 13, 2019
@rwinch
Copy link
Member

rwinch commented May 13, 2019

I've commented on the issue you linked to. Let's discuss there before we look at the PR

@rwinch rwinch self-assigned this May 20, 2019
@rwinch rwinch added status: declined A suggestion or change that we don't feel we should currently apply and removed status: waiting-for-feedback We need additional information before we can continue labels May 20, 2019
@rwinch rwinch closed this May 20, 2019
@rwinch
Copy link
Member

rwinch commented May 20, 2019

Thank you for the PR @ch4mpy! I'm closing this as decline. Please see #6857 (comment) for the reasoning

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: test An issue in spring-security-test status: declined A suggestion or change that we don't feel we should currently apply
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SecurityContextRequestPostProcessorSupport small-improvements
3 participants