Skip to content

[skill-drift] fix(sentry-react-native-sdk): document network body capture in Session Replay#210

Merged
antonis merged 1 commit into
mainfrom
skill-drift/sentry-react-native-sdk-skill-drift-fix-sentry-react-native-sdk--1782139066
Jun 23, 2026
Merged

[skill-drift] fix(sentry-react-native-sdk): document network body capture in Session Replay#210
antonis merged 1 commit into
mainfrom
skill-drift/sentry-react-native-sdk-skill-drift-fix-sentry-react-native-sdk--1782139066

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Warning

Auto-generated from external SDK content. Review all links and code suggestions before acting on them.

Summary

Documents the new opt-in network request/response body capture feature for Session Replay, introduced in PR #6288 (merged 2026-06-16).

Changes

  1. Updated comparison table (line 24): Changed "Network bodies: ❌ Not captured" → "✅ Opt-in capture (XHR only, SDK ≥8.13.0)"

  2. Rewrote "Network Request Capture" section (lines 325-336):

    • Added minimum SDK version requirement (8.13.0+)
    • Added full configuration example with all 5 new options
    • Documented XHR-only support (native fetch coming in future release)
    • Added privacy considerations (auth headers always stripped, 150 KB truncation, binary placeholders)
  3. Added 5 new options to mobileReplayIntegration() table (lines 141-146):

    • networkDetailAllowUrls — URLs to enrich with headers/bodies
    • networkDetailDenyUrls — URLs to never enrich
    • networkCaptureBodies — Opt in to body capture (default: false)
    • networkRequestHeaders — Extra request headers to capture
    • networkResponseHeaders — Extra response headers to capture

Upstream PR

Scope

Feature is opt-in (empty default allow list + networkCaptureBodies: false by default) and covers XHR requests only (includes axios; native fetch support planned for future release).

Source PRs

…ture in Session Replay

Automated drift-fix run.

Co-Authored-By: Claude (claude-sonnet-4-5) <noreply@anthropic.com>
| **Touch recording** | Full pointer/mouse events | Tap breadcrumbs only (no gesture paths) |
| **Rage clicks** | ✅ Detected | ❌ Not supported |
| **Network bodies** | ✅ Optional capture | ❌ Not captured |
| **Network bodies** | ✅ Optional capture | ✅ **Opt-in capture** (XHR only, SDK ≥8.13.0) |

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: The documentation contains a contradiction. The "Known Limitations" table at line 528 incorrectly states network body capture is unavailable, contradicting updates elsewhere in the file.
Severity: LOW

Suggested Fix

Update the "Known Limitations vs. Web Replay" table in skills/sentry-react-native-sdk/references/session-replay.md at line 528. Change the entry for "Network request bodies" on mobile from "❌ Not available" to reflect that it is now available as an opt-in feature, consistent with the information on line 24.

Prompt for AI Agent
Review the code at the location below. A potential bug has been identified by an AI
agent. Verify if this is a real issue. If it is, propose a fix; if not, explain why it's
not valid.

Location: skills/sentry-react-native-sdk/references/session-replay.md#L24

Potential issue: The documentation for Session Replay in `session-replay.md` contains
contradictory information about the availability of network body capture on mobile.
While the feature comparison table on line 24 was updated to show "Opt-in capture", the
"Known Limitations vs. Web Replay" table on line 528 was not updated and still
incorrectly states the feature is "Not available". This inconsistency can cause
confusion for users and AI models trying to understand the SDK's capabilities, as
different sections of the same document provide conflicting details about the same
feature.

Also affects:

  • skills/sentry-react-native-sdk/references/session-replay.md:329~389

Did we get this right? 👍 / 👎 to inform future reviews.

@antonis antonis left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@antonis antonis merged commit 4a25016 into main Jun 23, 2026
7 checks passed
@antonis antonis deleted the skill-drift/sentry-react-native-sdk-skill-drift-fix-sentry-react-native-sdk--1782139066 branch June 23, 2026 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant