getsentry / getsentry/sentry-react-native

Migrate route param gating to `dataCollection.userInfo`

Ouverte
#5,998 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub
Errors React-Native Task
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

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.