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

🎉 implement vulnid logic for suse #11890

Merged
merged 1 commit into from
Feb 26, 2025

Conversation

manuel-sommer
Copy link
Contributor

No description provided.

@github-actions github-actions bot added settings_changes Needs changes to settings.py based on changes in settings.dist.py included in this PR ui labels Feb 24, 2025
Copy link

DryRun Security Summary

Added SUSE-SU vulnerability URL mapping and updated URL generation functionality to support SUSE security advisories with proper input handling.

Expand for full summary

The PR adds a new SUSE-SU vulnerability URL mapping in settings and updates the vulnerability URL generation function to support SUSE-SU identifiers. No security vulnerabilities were found in these changes. The modifications enhance URL generation for SUSE security advisories with input sanitization and consistent URL formatting.

Code Analysis

We ran 9 analyzers against 2 files and 1 analyzer had findings. 8 analyzers had no findings.

Analyzer Findings
Configured Codepaths Analyzer 1 finding

Overall Riskiness

🔴 Risk threshold exceeded.

We've notified @mtesauro, @grendel513.

View PR in the DryRun Dashboard.

Copy link
Contributor

@mtesauro mtesauro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved

@mtesauro mtesauro merged commit 2c9a32f into DefectDojo:bugfix Feb 26, 2025
71 of 72 checks passed
@manuel-sommer manuel-sommer deleted the suse_vulnid branch February 26, 2025 19:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
settings_changes Needs changes to settings.py based on changes in settings.dist.py included in this PR ui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants