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

fix 10221 - 403 issue; authentication object is obtained from Security Context. #10267

Merged

Conversation

jagnathan
Copy link
Collaborator

fix #10221 - 403 issue; swagger annotations cause issue in authentication object. If user authorization is enabled, authentication object is obtained from SecurityContextHolder

@jagnathan jagnathan changed the title fix 10221 - 403 issue fix 10221 - 403 issue; authentication object is obtained from Security Context. Jul 11, 2023
@jagnathan jagnathan requested a review from dippindots July 11, 2023 21:41
fix 10221 - 403 issue; swagger annotations cause issue in authentication object. If user authorization is enabled, authentication object is obtained from SecurityContextHolder
@dippindots dippindots force-pushed the fix10221-403-authorized-studies-2 branch from cf2e30b to 660ed0a Compare July 17, 2023 15:03
@sonarcloud
Copy link

sonarcloud bot commented Jul 17, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@dippindots
Copy link
Member

Merging this pull request, I will test localdb tests in a new frontend release pr.

@dippindots dippindots added the bug label Jul 18, 2023
@dippindots dippindots merged commit 9e90a2a into cBioPortal:master Jul 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

403 on tags/fetch when show_unauthorized_studies=true
2 participants