Skip to content

Added ssrf utils file to check urls and applied it to risk recon parser#14631

Open
Jino-T wants to merge 4 commits intoDefectDojo:bugfixfrom
Jino-T:hackerone-3638741
Open

Added ssrf utils file to check urls and applied it to risk recon parser#14631
Jino-T wants to merge 4 commits intoDefectDojo:bugfixfrom
Jino-T:hackerone-3638741

Conversation

@Jino-T
Copy link
Copy Markdown
Contributor

@Jino-T Jino-T commented Apr 2, 2026 •
edited by Maffooch
Loading

  • Add a shared utility module for URL validation and normalization
  • Update the Risk Recon parser to use the new centralized URL handling
  • Add unit tests for the new utility module and updated parser behavior

@Maffooch Maffooch force-pushed the hackerone-3638741 branch from 2110797 to deedd7d Compare April 14, 2026 21:32
@Maffooch Maffooch changed the base branch from dev to bugfix April 14, 2026 21:32
@Jino-T Jino-T force-pushed the hackerone-3638741 branch from deedd7d to 6e58a8f Compare April 14, 2026 21:48
@Jino-T Jino-T marked this pull request as ready for review April 14, 2026 21:49
@Maffooch Maffooch added this to the 2.57.2 milestone Apr 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Maffooch Maffooch Maffooch approved these changes

@mtesauro mtesauro Awaiting requested review from mtesauro mtesauro is a code owner

@valentijnscholten valentijnscholten Awaiting requested review from valentijnscholten

@paulOsinski paulOsinski Awaiting requested review from paulOsinski

@blakeaowens blakeaowens Awaiting requested review from blakeaowens

At least 4 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

parser unittests

Projects

None yet

Milestone

2.57.2

Development

Successfully merging this pull request may close these issues.

2 participants

@Jino-T @Maffooch