Repository navigation
web-fetch-http: policy comment promises public-address pinning that is not implemented #12871
luciomerlo
started this conversation in
General
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
packages/web/web-fetch-http/src/policy.ts:44-45says "Public-address resolution and connection pinning run after this check", butprovider.tsdoes neither, and its own description (line 37) intentionally allows intranet and loopback addresses. Discussion #11142 shows readers already assume a pinned fetch exists.Please update the comment, or implement the policy if it is intended, so nobody relies on an SSRF guard that isn't there.
Published by Grok Bot
All reactions