-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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(oauth2): google oauth2 whitelisting. #11372
Conversation
DryRun Security SummaryThis pull request enhances the security of the Google OAuth2 authentication functionality in the DefectDojo application by adding support for configuring environment variables to specify whitelisted domains and emails, and updating the configuration file to properly handle the whitelisted values. Expand for full summarySummary: The code changes in this pull request focus on enhancing the security of the Google OAuth2 authentication From an application security perspective, these changes are positive as they allow for more granular control Files Changed:
Overall, these code changes appear to be security-focused improvements that enhance the security of the Code AnalysisWe ran
|
912116c
to
a991034
Compare
0f2e632
to
da7fbdb
Compare
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
The variable is a tuple and should be managed as such Signed-off-by: Julien Godin <julien.godin@camptocamp.com>
da7fbdb
to
aad7948
Compare
Conflicts have been resolved. A maintainer will review the pull request shortly. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved
* Update release step3 * fix(oauth2): Fix google oauth2 whitelisting. The variable is a tuple and should be managed as such Signed-off-by: Julien Godin <julien.godin@camptocamp.com> --------- Signed-off-by: Julien Godin <julien.godin@camptocamp.com> Co-authored-by: Cody Maffucci <46459665+Maffooch@users.noreply.github.com>
Fix google oauth2 whitelisting.
The variable is a tuple and should be managed as such
Description
Fixes: #6748
What
Google oauth using environment variables as configuration (k8s for instance) could not get the correct configuration.
The google whitelisting is an array and was red as a string.
It was hence not working the way it was intended to.
How
I used the same method used as the allowed_hosts to pars arrays and updated the documentation to explain how to use the configuration keys in a helm deployments.
Checklist
This checklist is for your information.
bugfix
branch.