-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
feat(remix): Skip span creation for OPTIONS
and HEAD
requests.
#11149
Conversation
size-limit report 📦
|
@onurtemizkan just before I merge this, would you mind explaining why these are the requests mentioned in the issue/discord? I'm probably missing some context here and I'm curious how you identified it. |
@Lms24, looking again, I'm now not sure if this completely resolves all the cases mentioned in the linked issue. That's a misunderstanding on my side. I have seen a polluted transaction list caused by I'm converting the PR to draft now, I'll try to experiment with the external requests and update the patch accordingly. Thanks for the review! |
Ok makes sense, thanks for double checking! |
d984176
to
de88797
Compare
de88797
to
2f2f831
Compare
Do we want to backport to v7? |
Yes, I think so, I'll open a PR. |
…etsentry#11149) Fixes: getsentry#11105 Skips span creation for `OPTIONS` and `HEAD` requests in `wrapRequestHandler`, which apparently are the unwanted requests mentioned in getsentry#11105. We can also implement a more precise filtering but this seems to resolve it on my local test application.
Fixes: #11105
Skips span creation for
OPTIONS
andHEAD
requests inwrapRequestHandler
, which apparently are the unwanted requests mentioned in #11105. We can also implement a more precise filtering but this seems to resolve it on my local test application.