-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
fix(replay): Fix missing fetch/xhr requests #7134
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
Conversation
Logic was flipped for the filter function so fetch and xhr requests were not being recorded at all.
We should add an integration test for this. |
size-limit report 📦
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Damn, good catch! filter boolean responses are always a bit ambiguous 😬
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch!
I added a point to our list of integration test scenarios: #7044
Replay SDK metrics 🚀
develop |
Revision | LCP | CLS | CPU | JS heap avg | JS heap max | netTx | netRx | netCount | netTime |
---|---|---|---|---|---|---|---|---|---|
b49d9f7 | +52.79 ms | -0.00 ms | +15.46 pp | +1.07 MB | +2.21 MB | +2.59 kB | +41 B | +1 | +70.48 ms |
7e57cb7 | +47.89 ms | +0.00 ms | +10.59 pp | +1.09 MB | +2.21 MB | +2.59 kB | +41 B | +1 | +72.93 ms |
538c3a6 | +63.63 ms | +0.00 ms | +14.14 pp | +1.07 MB | +2.16 MB | +2.6 kB | +41 B | +1 | +112.08 ms |
fc7b716 | +52.44 ms | +0.00 ms | +14.74 pp | +1.09 MB | +2.21 MB | +2.68 kB | +41 B | +1 | +89.64 ms |
Last updated: Fri, 10 Feb 2023 10:44:36 GMT
Logic was flipped for the filter function so fetch and xhr requests were not being recorded at all.
Closes https://github.com/getsentry/team-replay/issues/9
Closes #7078