Skip to content

tel:-links should probably be scrubbed #16175

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
3 tasks done
jgarplind opened this issue Apr 30, 2025 · 2 comments
Open
3 tasks done

tel:-links should probably be scrubbed #16175

jgarplind opened this issue Apr 30, 2025 · 2 comments

Comments

@jgarplind
Copy link

Is there an existing issue for this?

How do you use Sentry?

Sentry Saas (sentry.io)

Which SDK are you using?

@sentry/react

SDK Version

9.10.1

Framework Version

18.3.1

Link to Sentry event

No response

Reproduction Example/SDK Setup

No response

Steps to Reproduce

User "dead clicked" a tel:-link, containing a real phone number.

Expected Result

The phone number should be scrubbed, since phone numbers are generally PII(?)

I understand that we can manually do this scrubbing in beforeSend, but it seems like a generic issue.

Actual Result

Image

@jgarplind jgarplind added the Bug label Apr 30, 2025
@getsantry getsantry bot moved this to Waiting for: Product Owner in GitHub Issues with 👀 3 Apr 30, 2025
@github-actions github-actions bot added the React label Apr 30, 2025
@jgarplind jgarplind changed the title tel:-links should be scrubbed tel:-links should probably be scrubbed Apr 30, 2025
@chargome
Copy link
Member

@jgarplind thanks for reporting this one as well.

@s1gr1d one more for PII

@mydea
Copy link
Member

mydea commented May 2, 2025

This is a bit trickier because it makes masking dependent on the attribute value 🤔 But it should be doable in maskAttribute, if it is an <a> element and a href attribute to check if it starts with email or tel to mask it 🤔

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

No branches or pull requests

3 participants