getsentry / getsentry/sentry-javascript
`tel:`-links should probably be scrubbed
- 主要言語
- TypeScript
- スター
- 8.7k
- フォーク
- 1.8k
- 平均マージ
- 1日 17時間
- マージ済み PR(30日)
- 523
説明
### Is there an existing issue for this?
- [x] I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
- [x] I have reviewed the documentation https://docs.sentry.io/
- [x] I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases
### 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

コントリビューションガイド
調査の方向性
まず、@sentry/react 9.10.1 でデッドクリックのシナリオを再現し、レポートに示されているキャプチャされたイベントを調査します。tel: リンクの URL がどのようにイベントデータに入るかを追跡し、beforeSend を必要とせずに電話番号がスクラブされることを確認します。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- react, typescript
- 領域
- frontend, security
- issue の種類
- バグ
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 35/100