getsentry / getsentry/sentry-react-native
Migrate route param gating to `dataCollection.userInfo`
- Vorherrschende Sprache
- TypeScript
- Sterne
- 1.8k
- Forks
- 366
- Ø Merge
- 1 T. 5 Std.
- Gemergte PRs (30 T.)
- 93
Beschreibung
Replace `sendDefaultPii` check in React Navigation integration with `dataCollection.userInfo`.
## Callsite
* `reactnavigation.ts:488` — dynamic route params (e.g. `[id]`, `[...slug]`) are only attached to navigation spans when `sendDefaultPii` is true
Should read from resolved `dataCollection.userInfo` instead.
## References
* `packages/core/src/js/tracing/reactnavigation.ts`
* `packages/core/test/tracing/reactnavigation.test.ts` — existing tests for route param behavior
Beitragsleitfaden
Rechercherichtung
Beginne in packages/core/src/js/tracing/reactnavigation.ts bei der Verarbeitung der Routenparameter etwa in Zeile 488 und lies anschließend die vorhandenen Tests für Routenparameter in packages/core/test/tracing/reactnavigation.test.ts. Aktualisiere das Gate-Verhalten so, dass die aufgelösten dataCollection.userInfo verwendet werden, und führe die Tracing-Tests für React Navigation aus, um zu bestätigen, dass dynamische Routenparameter wie erwartet funktionieren.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- react-native, typescript
- Bereich
- mobile
- Issue-Typ
- Refactoring
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Ruhig
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 58/100