Skip to content

Refactor DNS result inspection#743

Open
timokoessler wants to merge 1 commit intomainfrom
refactor-inspect-dns
Open

Refactor DNS result inspection#743
timokoessler wants to merge 1 commit intomainfrom
refactor-inspect-dns

Conversation

@timokoessler
Copy link
Copy Markdown
Member

Remove double stored SSRF check through refactoring

Comment thread library/vulnerabilities/ssrf/inspectDNSLookupCalls.ts
@codecov
Copy link
Copy Markdown

codecov Bot commented Sep 16, 2025

Codecov Report

❌ Patch coverage is 97.87234% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...rary/vulnerabilities/ssrf/inspectDNSLookupCalls.ts 97.87% 2 Missing ⚠️

📢 Thoughts on this report? Let us know!

@timokoessler
Copy link
Copy Markdown
Member Author

Reason for the missing lines in Code Coverage is that the redirection protection is still disabled.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant