rrweb-io / rrweb

record and replay the web
https://www.rrweb.io/
MIT License
16.84k stars 1.44k forks source link

[Feature Request]: Addition of flag maskAllText #1563

Open chetan-187 opened 2 months ago

chetan-187 commented 2 months ago

Preflight Checklist

What package is this feature request for?

rrweb

Problem Description

If I want to mask all text on the DOM it takes me to implement a function and do additional efforts. The addition of maskAllText flag with make it as is as it does for maskAllInputs

Proposed Solution

Solution is I should be able to pass a option maskAllText while initializing rrweb, and it shoud replace all text from DOM with *

Alternatives Considered

The alternative feature is implementing maskInputFn

Additional Information

This feature would be really helpful as to mask the DOM data is sometimes need for financial or security businesses.

kmlwlkwk commented 2 months ago

We did implementation of masking all text but it might be very expensive and you need to be very careful

chetan-187 commented 2 months ago

Hey @kmlwlkwk, can you please explain in which sense it will be expensive and what are the critical things?