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.

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

2.x: Security context not overridden #7511

Merged

Conversation

Verdent
Copy link
Member

@Verdent Verdent commented Sep 4, 2023

Description

Security context is no longer overridden when combining WebSecurity and JAX-RS endpoint. This issue was happening in SecurityPreMatchingFilter in cases where WebSecurity authenticated user. SecurityPreMatchingFilter obtained already processed SecurityContext and replaced it with not processed one.

Fixes #6414

Documentation

None

Signed-off-by: David Kral <david.k.kral@oracle.com>
@Verdent Verdent requested a review from tomas-langer September 4, 2023 12:23
@Verdent Verdent self-assigned this Sep 4, 2023
@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Sep 4, 2023
Signed-off-by: David Kral <david.k.kral@oracle.com>
@Verdent Verdent changed the title Security context not overridden 2.x: Security context not overridden Sep 4, 2023
Signed-off-by: David Kral <david.k.kral@oracle.com>
@Verdent Verdent merged commit deb4854 into helidon-io:helidon-2.x Sep 5, 2023
@Verdent Verdent deleted the 6414-security-context-not-overridden branch September 5, 2023 10:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCA Verified All contributors have signed the Oracle Contributor Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SecurityPreMatchingFilter is overriding the SecurityContext when using webserver security
2 participants