Skip to content

update invicti parser to use FirstSeenDate#14610

Merged
mtesauro merged 6 commits intoDefectDojo:bugfixfrom
paulOsinski:invicti
Apr 13, 2026
Merged

update invicti parser to use FirstSeenDate#14610
mtesauro merged 6 commits intoDefectDojo:bugfixfrom
paulOsinski:invicti

Conversation

@paulOsinski
Copy link
Copy Markdown
Contributor

Invicti parser will now apply FirstScanDate, if present, to the Finding's date object. This change does not affect deduplication or anything destructive, we're simply parsing new data that we weren't before.

@Maffooch Maffooch modified the milestones: 2.57.0, 2.57.1 Apr 3, 2026
@paulOsinski
Copy link
Copy Markdown
Contributor Author

paulOsinski commented Apr 7, 2026

@Maffooch I have added the flag gating for USE_FIRST_SEEN. If you're wondering about the force-push, I made a change to the Checkmarx parser but realized it was out of scope so I dropped it from the PR.

@paulOsinski paulOsinski requested a review from Maffooch April 7, 2026 19:11
Copy link
Copy Markdown
Member

@valentijnscholten valentijnscholten left a comment

Choose a reason for hiding this comment

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

Looks good, but ideally we update the comment on the USE_FIRST_SEEN flag in settings.dist.py as it currently only mentions Qualys. Maybe also add a line to the docs for both parsers supporting this flag? Not blocking for merge.

Copy link
Copy Markdown
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 fdcdc34 into DefectDojo:bugfix Apr 13, 2026
157 checks passed
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