getsentry / getsentry/sentry-react-native

Migrate route param gating to `dataCollection.userInfo`

オープン
#5,998 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る
Errors React-Native Task
主要言語
TypeScript
スター
1.8k
フォーク
366
平均マージ
1日 6時間
マージ済み PR(30日)
94

説明

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

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

packages/core/src/js/tracing/reactnavigation.ts の 488 行目付近にあるルートパラメーターの処理から始め、次に packages/core/test/tracing/reactnavigation.test.ts にある既存のルートパラメーターのテストを確認してください。解決済みの dataCollection.userInfo を使用するようにゲーティング動作を更新し、React Navigation の tracing テストを実行して、動的なルートパラメーターが期待どおりに動作することを確認してください。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
react-native, typescript
領域
mobile
issue の種類
リファクタリング
難易度
2/5
見積もり時間
1〜3時間
活発さ
静か
明瞭さ
明確に書かれている
初心者へのやさしさ
58/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。