-
Couldn't load subscription status.
- Fork 1.8k
Release: Merge back 2.47.1 into dev from: master-into-dev/2.47.1-2.48.0-dev #12573
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
Conversation
….48.0-dev Release: Merge back 2.47.0 into bugfix from: master-into-bugfix/2.47.0-2.48.0-dev
* Update pro_features.md Add Deduplication Tuning section and link to X-tool deduplication screenshot. * Deduplication X tool Deduplication image image showing cross-tool deduplication screenshot * Initial version - Deduplication Tuning documentation of deduplication tuning * Deduplication Settings Menu * Additional screenshots for deduplication tuning Same tool, cross tool, reimport deduplication screenshots with menu selection. * Update tune_deduplication.md Added additional information about retroactive execution of deduplication adjustments. * Update pro_features.md Cleaned up language about deduplication, increase technical specificity + less markety. Added newlines for "See our guide for more information" . Some had it / others didn't - easier navigation visually & to links for more info. * Update docs/content/en/working_with_findings/finding_deduplication/tune_deduplication.md * We love the Oxford comma :) Co-authored-by: Harold Blankenship <[email protected]> --------- Co-authored-by: Cody Maffucci <[email protected]> Co-authored-by: Harold Blankenship <[email protected]>
* update changelog 2.47.0 * change article titles * change headers for sidebar --------- Co-authored-by: Paul Osinski <[email protected]>
* SAML Login: Respect `next` parameter When logging in with SAML, the `next` parameter is not respected like it is for other SSO providers * Add pre-wrapper * Add trailing name
Release: Merge release into master from: release/2.47.1
🔴 Risk threshold exceeded.This pull request contains an open redirect vulnerability in the SAML2 login link that could allow attackers to redirect users to arbitrary external sites, and includes sensitive edits to base and login template files that may require additional review or configuration in
|
| Vulnerability | Open Redirect |
|---|---|
| Description | The SAML2 login link constructs a redirect URL using unvalidated user input from request.GET.next. This allows an attacker to potentially redirect users to an arbitrary external site after login, which could be used for phishing or other malicious purposes. Best practices recommend validating and sanitizing redirect URLs, preferably using an allow-list of trusted destinations. |
django-DefectDojo/dojo/templates/dojo/login.html
Lines 97 to 103 in a38f8cd
| {% if SAML2_ENABLED is True %} | |
| <div class="col-sm-offset-1 col-sm-2"> | |
| <a id="oauth-login-saml" rel="nofollow" data-method="post" href="/saml2/login?next={{ request.GET.next }}" style="color: rgb(255,255,255)" class="btn btn-success" type="button">{{ SAML2_LOGIN_BUTTON_TEXT }}</a> | |
| </div> | |
| {% endif %} | |
| </div> |
⚠️ Configured Codepaths Edit in dojo/templates/base.html
| Vulnerability | Configured Codepaths Edit |
|---|---|
| Description | Sensitive edits detected for this file. Sensitive file paths and allowed authors can be configured in .dryrunsecurity.yaml. |
⚠️ Configured Codepaths Edit in dojo/templates/base.html
| Vulnerability | Configured Codepaths Edit |
|---|---|
| Description | Sensitive edits detected for this file. Sensitive file paths and allowed authors can be configured in .dryrunsecurity.yaml. |
⚠️ Configured Codepaths Edit in dojo/templates/dojo/login.html
| Vulnerability | Configured Codepaths Edit |
|---|---|
| Description | Sensitive edits detected for this file. Sensitive file paths and allowed authors can be configured in .dryrunsecurity.yaml. |
We've notified @mtesauro.
All finding details can be found in the DryRun Security Dashboard.
|
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
|
Conflicts have been resolved. A maintainer will review the pull request shortly. |
Release triggered by
rossops