Skip to content

docs: Update DrHeader parser documentation#13685

Closed
skywalke34 wants to merge 1 commit intoDefectDojo:devfrom
skywalke34:update-drheader-docs
Closed

docs: Update DrHeader parser documentation#13685
skywalke34 wants to merge 1 commit intoDefectDojo:devfrom
skywalke34:update-drheader-docs

Conversation

@skywalke34
Copy link
Copy Markdown
Contributor

Description

update drheader json parser documentation

Test results
Tested review in hugo

Documentation

drheader parser documenatation

@@ -1,15 +1,185 @@
---
title: "DrHeader"
title: "DrHeader JSON Importer"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

If you change the title, you also have to change the name within the parser itself and also then, please create a db migrations file, otherwise unittests fail. So, I would suggest to leave the title as it is.

Also, you can target this PR against bugfix to get it live on production faster.

@Maffooch Maffooch requested a review from paulOsinski November 26, 2025 16:15
@Maffooch
Copy link
Copy Markdown
Contributor

@paulOsinski can you please assist @skywalke34 with this PR?


| Finding Field | Default Value | Parser Line # | Notes |
|---------------|---------------|---------------|-------|
| static_finding | False | 37 | All findings explicitly marked as dynamic (runtime scanner) |
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

for similar reasons to what was discussed on your other PR I think we need to hold on this one until we can validate the field mappings for accuracy.

@paulOsinski
Copy link
Copy Markdown
Contributor

Closing this one

@paulOsinski paulOsinski closed this Dec 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants