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

Django Admin pages for editing Notifications #9193

Merged
merged 3 commits into from Dec 22, 2023

Conversation

tomaszn
Copy link
Contributor

@tomaszn tomaszn commented Dec 19, 2023

Description

This customizes the built-in (and enabled by default) Django Admin interface, and adds a visual way to add, remove, and modify notifications in DefectDojo.

This is a workaround for #9192.

This is how the notification list will look like:

Screenshot_2023-12-19_13-15-12

This is how the a single notification editor will look like:

Screenshot_2023-12-19_13-21-51

Test results

Tested manually.

Documentation

Django Admin functionality is not described in the documentation.

Copy link

dryrunsecurity bot commented Dec 19, 2023

Contextual Security Analysis

As DryRun Security performs checks, we’ll summarize them here. You can always dive into the detailed results in the section below for checks.

Status DryRun Security Check
AI-powered Sensitive Function Check
Configured Sensitive Files Check
AI-powered Sensitive Files Check

Chat with your AI-powered Security Buddy by typing @dryrunsecurity followed by your question into a comment.
Example: @dryrunsecurity What are common security issues with web application cookies?

Install and configure more repositories at DryRun Security

dojo/models.py Outdated Show resolved Hide resolved
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

@Maffooch Maffooch merged commit 251dab3 into DefectDojo:dev Dec 22, 2023
120 of 121 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants