-
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: slack address on issue template #10411
Conversation
Hi there 👋, @DryRunSecurity here, below is a summary of our analysis and findings.
Note 🟢 Risk threshold not exceeded. Change Summary (click to expand)The following is a summary of changes in this pull request made by me, your security buddy 🤖. Note that this summary is auto-generated and not meant to be a definitive list of security issues but rather a helpful summary from a security perspective. Summary: The changes in this GitHub pull request update the link for the OWASP Slack workspace invitation in the The bug report template is a common and recommended practice to help users report issues effectively and provide the necessary information for the development team to investigate and resolve the problems. However, it's worth noting that the template does not explicitly mention any security-related information or vulnerabilities. If users were to report security vulnerabilities, it would be important to have a separate security-focused issue template or a clear process for handling such reports, potentially through a responsible disclosure program or a dedicated security contact. Files Changed:
Powered by DryRun Security |
@fcecagno This is a great change - thanks for noticing and doing the PR but we only take PRs against the dev or bugfix branch. Details are at https://github.com/DefectDojo/django-DefectDojo/blob/master/readme-docs/CONTRIBUTING.md Please change this PR to the bugfix branch so it can get approved and merged. (besides targeting master, this looks fine) Thanks! |
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.
Helm test is failing because this PR targeted master initially. The failure can be disregarded in this case since there is not application code being touched
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
Description
Address to Slack in the bug report template doesn't exist anymore, so this PR updates it to the right link.