Skip to content

feat(apollo): [Data Collection 8] Apply outgoing request body policy#5804

Draft
adinauer wants to merge 2 commits into
feat/data-collection-incoming-response-bodyfrom
feat/data-collection-outgoing-request-body
Draft

feat(apollo): [Data Collection 8] Apply outgoing request body policy#5804
adinauer wants to merge 2 commits into
feat/data-collection-incoming-response-bodyfrom
feat/data-collection-outgoing-request-body

Conversation

@adinauer

@adinauer adinauer commented Jul 21, 2026

Copy link
Copy Markdown
Member

PR Stack (Data Collection)


📜 Description

Apply the outgoing request body policy to Apollo 3 and Apollo 4 failed GraphQL events.

When body collection is enabled, GraphQL document and variable policies continue to filter their respective fields. Disabling body content retains request body size and other technical metadata. Existing always-collect behavior remains unchanged when Data Collection is absent.

💡 Motivation and Context

Apollo’s serialized GraphQL request is outgoing HTTP body content and must honor both its directional body policy and its structured GraphQL controls.

Refs #5666

💚 How did you test it?

  • ./gradlew spotlessApply apiDump
  • Apollo 3 and Apollo 4 failed-request tests

📝 Checklist

  • I added GH Issue ID & Linear ID
  • I added tests to verify the changes.
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • Review from the native team if needed.
  • No breaking change or entry added to the changelog.
  • No breaking change for hybrid SDKs or communicated to hybrid SDKs.

🔮 Next steps

Apply the outgoing server response body policy and HTTP header policies.

#skip-changelog

⚠️ Merge this PR using a merge commit (not squash). Only the collection branch is squash-merged into main.

Use the Data Collection outgoing request body decision when attaching failed GraphQL request content in Apollo 3 and 4. Retain request body size metadata and continue applying GraphQL document and variable controls when body collection is enabled.

Co-Authored-By: Claude <noreply@anthropic.com>
@github-actions

github-actions Bot commented Jul 21, 2026

Copy link
Copy Markdown
Contributor
Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against 04f91e1

@sentry

sentry Bot commented Jul 21, 2026

Copy link
Copy Markdown

📲 Install Builds

Android

🔗 App Name App ID Version Configuration
SDK Size io.sentry.tests.size 8.48.0 (1) release

⚙️ sentry-android Build Distribution Settings

Bring the queue option removal and preceding stack updates into this layer.
@github-actions

Copy link
Copy Markdown
Contributor

Performance metrics 🚀

  Plain With Sentry Diff
Startup time 316.59 ms 369.54 ms 52.95 ms
Size 0 B 0 B 0 B

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.

1 participant