Skip to content

update invicti parser to use FirstSeenDate#14610

Open
paulOsinski wants to merge 6 commits intoDefectDojo:bugfixfrom
paulOsinski:invicti
Open

update invicti parser to use FirstSeenDate#14610
paulOsinski wants to merge 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.

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.

3 participants