-
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
add engagement closed MS teams, Email, Alert, and Slack template #11204
Conversation
DryRun Security SummaryThe pull request updates the notification templates for the DefectDojo application, focusing on improving the security, internationalization, and localization of the Microsoft Teams, Slack, and email notifications sent when an engagement is closed. Expand for full summarySummary: The code changes in this pull request are focused on updating the notification templates for the DefectDojo application. The changes include updates to the Microsoft Teams, Slack, and email notification templates that are used to inform users when an engagement is closed. From an application security perspective, the changes do not introduce any obvious security vulnerabilities. The templates are using secure templating systems, such as Jinja2 and Django's template language, which help prevent common web application vulnerabilities like cross-site scripting (XSS). The templates also include features for internationalization and localization, as well as the ability to display system-level disclaimers. The key security-related aspects of the changes include:
Overall, the changes appear to be focused on improving the notification functionality of the DefectDojo application, with a strong emphasis on security and internationalization/localization considerations. Files Changed:
Code AnalysisWe ran Riskiness🟢 Risk threshold not exceeded. |
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
I suppose it might be a good idea to add it to all notification processors (alerts, slack, mail). |
@kiblik thank you for calling this out! I agree with that move |
Add a MS Teams, Email, Alert, and Slack template for engagement closed.
[sc-6170]