Skip to content

Mark request URIs and form file paths as Psalm taint sinks - #402

Open
danog wants to merge 1 commit into
amphp:5.xfrom
danog:taint-annotations
Open

danog wants to merge 1 commit into
amphp:5.xfrom
danog:taint-annotations

Conversation

@danog

@danog danog commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

Psalm doesn't analyse vendor code, so its taint analysis can't see that a user-controlled URI ends up in an outgoing request. This marks the $uri of Request::__construct() and Request::setUri() as ssrf sinks, and the local $path of Form::addFile() as a file sink.

https://claude.ai/code/session_016B9dWRw6Lako9sXAWMtWAx

This branch has not been deployed

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant