feat(pii): add data processors that are shared within framework SDKs - #2195
feat(pii): add data processors that are shared within framework SDKs#2195Litarnus wants to merge 4 commits into
Conversation
![sentry[bot]](https://avatars.githubusercontent.com/in/12637?s=60&v=4){kind=small}
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 16ded29. Configure here.
| $body = $request->getParsedBody(); | ||
| if ($body !== null) { | ||
| return self::collect($options, 'incomingRequest', $body); | ||
| } |
There was a problem hiding this comment.
Empty parsed body skips stream
High Severity
collectServerRequest treats any non-null getParsedBody() result as the full body, including an empty array, and never reads the request stream. The default RequestFetcher builds requests with ServerRequest::fromGlobals(), which always sets the parsed body to $_POST. JSON and other non-form payloads leave $_POST empty, so those bodies are dropped and events can receive data: [] instead, including on GET requests.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit 16ded29. Configure here.
1 participant
Provides classes for HTTP data extraction which can be used in Symfony/Laravel without having to write them per SDK