Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Send request payload in Apollo integrations #2148

Open
adinauer opened this issue Jun 30, 2022 · 1 comment
Open

Send request payload in Apollo integrations #2148

adinauer opened this issue Jun 30, 2022 · 1 comment

Comments

@adinauer
Copy link
Member

adinauer commented Jun 30, 2022

Problem Statement

Currently the request payload is not sent by Apollo (v2 and v3) integrations. We should send it up to https://docs.sentry.io/platforms/java/configuration/options/#max-request-body-size

Solution Brainstorm

Take a look at other integrations that already send the request body.

@adinauer
Copy link
Member Author

adinauer commented Jul 27, 2022

Does PII stripping run on it?
Should it be guarded behind send default pii?

Created getsentry/team-mobile#41 to track across SDKs

Make sure it's documented in develop docs how integrations should send request body.

@adinauer adinauer moved this from Needs Discussion to Backlog in Mobile & Cross Platform SDK Jul 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Backlog
Development

No branches or pull requests

2 participants