getsentry / getsentry/sentry-react-native
Migrate route param gating to `dataCollection.userInfo`
- Langage dominant
- TypeScript
- Étoiles
- 1.8k
- Forks
- 367
- Merge moyen
- 1 j 6 h
- PR mergées (30 j)
- 94
Description
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
Guide de contribution
Ouvrir le guide de contribution
Piste de recherche
Commencez dans packages/core/src/js/tracing/reactnavigation.ts, au niveau de la gestion des paramètres de route autour de la ligne 488, puis consultez les tests existants des paramètres de route dans packages/core/test/tracing/reactnavigation.test.ts. Mettez à jour le comportement de gating afin d’utiliser les dataCollection.userInfo résolus, puis exécutez les tests de tracing de React Navigation pour confirmer que les paramètres de route dynamiques se comportent comme prévu.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- react-native, typescript
- Domaine
- mobile
- Type d'issue
- Refactorisation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Activité
- Calme
- Clarté
- Clairement spécifiée
- Accessibilité débutants
- 58/100