Skip to content

Comments

sla_deadline doesn't work with mitigated findings#8279

Merged
mtesauro merged 1 commit intoDefectDojo:bugfixfrom
coheigea:coheigea/sla-date
Jun 28, 2023
Merged

sla_deadline doesn't work with mitigated findings#8279
mtesauro merged 1 commit intoDefectDojo:bugfixfrom
coheigea:coheigea/sla-date

Conversation

@coheigea
Copy link
Contributor

@coheigea coheigea commented Jun 19, 2023

Description

The sla_deadline function doesn't work with mitigated findings, as it just applies sla days remaining to the current date, instead of the mitigated date.

Also I added a mitigated date for the Veracode SCA parser when findings are mitigated, as the date isn't contained in the JSON.

Test results

New tests added in PR.

@coheigea coheigea changed the base branch from master to bugfix June 19, 2023 15:16
@coheigea coheigea force-pushed the coheigea/sla-date branch from ae9db4d to 21671ef Compare June 19, 2023 15:18
@github-actions
Copy link
Contributor

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

@coheigea coheigea force-pushed the coheigea/sla-date branch from 21671ef to 4049214 Compare June 19, 2023 15:23
@github-actions
Copy link
Contributor

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

@coheigea coheigea force-pushed the coheigea/sla-date branch from 4049214 to 8f07053 Compare June 19, 2023 15:28
@coheigea
Copy link
Contributor Author

@Maffooch I think I accidentally submitted this originally against master. How can I force the checks to run again, now that I rebased against bugfix?

@Maffooch Maffooch closed this Jun 20, 2023
@Maffooch Maffooch reopened this Jun 20, 2023
@Maffooch
Copy link
Contributor

Testing on a different day to make sure results are consistent

@Maffooch Maffooch closed this Jun 21, 2023
@Maffooch Maffooch reopened this Jun 21, 2023
Copy link
Contributor

@Maffooch Maffooch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All green :)

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

@mtesauro mtesauro merged commit cc5a2c4 into DefectDojo:bugfix Jun 28, 2023
@coheigea coheigea deleted the coheigea/sla-date branch June 28, 2023 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants