getsentry / getsentry/sentry-javascript

Improve Redirect detection

Đang mở
#17,282 1 bình luận 0 reaction 1 người được giao Được @Lms24 nhận Xem trên GitHub
Browser Improvement State: Needs Discussion
Ngôn ngữ chính
TypeScript
Star
8.7k
Fork
1.8k
Merge trung bình
1 ngày 17 giờ
Pull request đã merge (30 ngày)
515

Mô tả

### Description

Our current redirection detection approach works in some cases but it fails in others, for example in the Sentry frontend (oh the irony 😅)

Some ideas to improve the detector:

* apply it generally to anyone calling `startBrowserTracingNavigationSpan`. Right now, we only apply redirect detection to the default browserTracingIntegration and any derivateives that rely on the default behaviour. Since in Sentry we use the React Router browserTracingIntegration, the logic is not applied.
* Increase time thresholds in general. 300ms is very short, especially for the first pageload
* Reset `lastInteractionTimestamp` when starting a new navigation *root* span. Previous interactions should not matter for subsequent navigations/redirects.
* Add separate time thresholds for
* pageload -> 1st navigation: A pageload likely takes some time for the router to initialize and potentially redirect. 300ms is *very* short here.
* navigation -> navigation: Here the time window likely doesn't need to be too long. 300ms is likely enough.
* For now, we can increase the threshold in general and re-evaluate if we get false positives for the navigation -> navigation case.

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.