getsentry / getsentry/sentry-react-native

[SR] Recursive unmasking

Aperta
#4,719 7 commenti 6 reazioni 0 assegnatari Vedi su GitHub
Explore > Replays Feature React-Native Replays
Lingua principale
TypeScript
Stelle
1.8k
Fork
366
Merge medio
1g 5h
PR unite (30g)
93

Descrizione

### Problem Statement

Currently, [unmasking a component](https://docs.sentry.io/platforms/react-native/session-replay/privacy/#general-masking-rules) affects only the component directly wrapped with ``. For some use cases where the actual text components are two or more levels down in a nested component tree, this can become cumbersome and may also require unwanted updates to component code (e.g. several levels into a Button component - user wants to unmask the button label).

### Solution Brainstorm

Provide an API to mark a component for unmasking recursively.

Note: There was a conscious decision not do to this initially, because it makes it easy to unmask more than intended. If this is implemented, it should be very clear to the user what happens, and very hard/impossible to do it unintentionally.

E.g. something like `` (default: false)

---

There is currently no priority for this item. If you are interested in this, please upvote the issue 👍

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia esaminando l’API Sentry.Unmask esistente e le regole di mascheramento React Native collegate. Definisci esplicitamente un opt-in ricorsivo e il comportamento dei suoi discendenti, quindi verifica che il testo annidato venga smascherato solo quando richiesto e che lo smascheramento ordinario rimanga invariato.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
react-native, typescript
Ambito
mobile
Tipo di issue
Funzionalità
Difficoltà
5/5
Tempo stimato
Più di una settimana
Stato di attività
Attiva
Chiarezza
Abbastanza chiara
Idoneità per principianti
35/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.