Skip to content

Fix stored SSRF detection when the hostname is the addresss - #231

Merged
bitterpanda63 merged 3 commits into
mainfrom
fix-stored-ssrf-detection-when-address-is-hostname
Jan 9, 2026
Merged

Fix stored SSRF detection when the hostname is the addresss#231
bitterpanda63 merged 3 commits into
mainfrom
fix-stored-ssrf-detection-when-address-is-hostname

Conversation

@marksmith

@marksmith marksmith commented Jan 9, 2026

Copy link
Copy Markdown
Collaborator

This change follows the lead of AikidoSec/firewall-python#554, and does not consider IMDS addresses as stored SSRF attacks when the hostname and the address are the same.

Unknown payload properties are no longer reported to Aikido Core so that they do not appear in the Aikido dashboard. This was suggested in the internal discussion for this issue.

Summary by Aikido

Security Issues: 0 🔍 Quality Issues: 2 Resolved Issues: 0

⚡ Enhancements

  • Omitted unknown payload properties from reports to avoid dashboard.

🐛 Bugfixes

  • Excluded addresses identical to hostname from stored SSRF detection.

More info

When the address is the hostname.
So that they are not displayed as "unknown" on in Aikido dashboard.
Comment thread lib/aikido/zen/scanners/stored_ssrf_scanner.rb
Comment thread lib/aikido/zen/scanners/stored_ssrf_scanner.rb Outdated
Comment thread lib/aikido/zen/scanners/stored_ssrf_scanner.rb
@codecov

codecov Bot commented Jan 9, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 80.00000% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
lib/aikido/zen/attack.rb 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@bitterpanda63
bitterpanda63 merged commit c40f054 into main Jan 9, 2026
33 of 34 checks passed
@bitterpanda63
bitterpanda63 deleted the fix-stored-ssrf-detection-when-address-is-hostname branch January 9, 2026 14:50
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.

2 participants