-
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
Docs: add supported report types index / maintenance #11921
Conversation
DryRun Security SummaryDefectDojo documentation repository updates revealed multiple security observations, including URL exposures, potential information disclosure, and configuration details that could aid security assessments. Expand for full summaryDefectDojo documentation repository received multiple updates across various files, primarily focusing on documentation link corrections, formatting improvements, and content clarifications. Multiple security observations were identified across the patches. Security Findings:
While no direct code-level vulnerabilities were introduced, the documentation updates provide insights into DefectDojo's internal structures and configuration methods that could be valuable for potential security assessments. |
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
@@ -141,7 +141,7 @@ When a user is removed from a given group in Azure AD, they will also be removed | |||
|
|||
### Open-Source | |||
|
|||
Open-Source users will need to map these variables in the local_settings.py file. (see [Configuration](../../open_source/installation/configuration)). | |||
Open-Source users will need to map these variables as an environment variable, or in the `local_settings.py` file. (see [Configuration](../../open_source/installation/configuration)). |
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.
"map" replaced with "set" elsewhere below -- does this one need to change as well?
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.
I am not actually sure what word is most accurate, map or set?
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
3653405
to
cd7b322
Compare
Conflicts have been resolved. A maintainer will review the pull request shortly. |
docs/content/en/customize_dojo/user_management/configure_sso.md
Outdated
Show resolved
Hide resolved
docs/content/en/customize_dojo/user_management/configure_sso.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Cody Maffucci <46459665+Maffooch@users.noreply.github.com>
[sc-10088]
[sc-10089]
[sc-10156]
[sc-10438]