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.

Sign up for GitHub

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

Jump to bottom

addition of sla expiration date field without signals #9356

Closed
wants to merge 29 commits into from
Closed

addition of sla expiration date field without signals #9356

wants to merge 29 commits into from

Conversation

blakeaowens
Copy link
Contributor

No description provided.

@blakeaowens
addition of sla expiration date field on the finding model
6983cde
@blakeaowens
add migration and fix indentation issue
aab6ccf
@blakeaowens
fix mitigated finding remaining sla days calculation
970fa5c
@blakeaowens
fix sla violation filter to return only active, sla violating findings
7d41feb
@blakeaowens
migration system settings fix
5f1b6b0
@blakeaowens
fix mitigation date vs datetime discrepancy
b7de4d8
@blakeaowens
fix breaking unit test
1360f21
@blakeaowens
move product save check to signal
b7dfc74
@blakeaowens
fix unit test failure
3431875
@blakeaowens
make signal operations async, fix sla config delete 500 error
4e51e5a
@blakeaowens
add unit tests to test sla expiration date functionality
b98f63a
@blakeaowens
restarting without signals
7d06020
@blakeaowens
add async updating flags, redo migration
e9f1deb
@blakeaowens
move signal logic to overriden save
b1d6a9a
@blakeaowens
fix errors for non-existing objects at creation
5c4fbc0
@blakeaowens
clean up comments and a few logical expressions
ec0af01
@github-actions github-actions bot added New Migration Adding a new migration file. Take care when merging. apiv2 unittests ui labels Jan 18, 2024
@dryrunsecurity DryRunSecurity
Copy link

dryrunsecurity bot commented Jan 18, 2024 •
edited
Loading

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
✅ Sensitive Functions Analyzer
❌ Configured Sensitive Files Check
✅ Sensitive Files Analyzer

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

@github-actions GitHub Actions
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions GitHub Actions
Copy link
Contributor

Conflicts have been resolved. A maintainer will review the pull request shortly.

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Feb 5, 2024

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@Maffooch Maffooch closed this Feb 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
apiv2 conflicts-detected New Migration Adding a new migration file. Take care when merging. ui unittests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@blakeaowens @Maffooch