Skip to content

remove Clair Klar Scan deduplication#9686

Merged
Maffooch merged 1 commit into
DefectDojo:devfrom
manuel-sommer:remove_unnecessary_dedup_setting
Mar 25, 2024
Merged

remove Clair Klar Scan deduplication#9686
Maffooch merged 1 commit into
DefectDojo:devfrom
manuel-sommer:remove_unnecessary_dedup_setting

Conversation

@manuel-sommer
Copy link
Copy Markdown
Contributor

@manuel-sommer manuel-sommer commented Mar 6, 2024

leftover from: #9355

@github-actions github-actions Bot added the settings_changes Needs changes to settings.py based on changes in settings.dist.py included in this PR label Mar 6, 2024
@dryrunsecurity
Copy link
Copy Markdown

dryrunsecurity Bot commented Mar 6, 2024

Hi there 👋, @DryRunSecurity here, below is a summary of our analysis and findings.

DryRun Security Status Findings
Sensitive Functions Analyzer 0 findings
Configured Sensitive Files Analyzer 0 findings
Sensitive Files Analyzer 1 findings

Note

🟢 Risk threshold not exceeded.

Tip

Get answers to your security questions. Add a comment in this PR starting with @DryRunSecurity. For example...

@dryrunsecurity What are common security issues with web application cookies?

Powered by DryRun Security

@manuel-sommer
Copy link
Copy Markdown
Contributor Author

Shall I write a unittest for such cases @Maffooch or is it too rare cover this case with a unittest?

Copy link
Copy Markdown
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

@Maffooch Maffooch merged commit 1f7c239 into DefectDojo:dev Mar 25, 2024
@manuel-sommer manuel-sommer deleted the remove_unnecessary_dedup_setting branch March 25, 2024 17:31
manuel-sommer added a commit to manuel-sommer/django-DefectDojo that referenced this pull request Mar 27, 2024
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants