-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
TypeError: Invalid URL is thrown from the Undici integration #8145
Comments
Hey @sbellone thanks for writing in! Crashing is really bad so we're on it to fix asap! What Node version are you on? Is there also a way to tell what the request is? Wondering why the URL is invalid here. Thanks! |
Thanks for your quick answer @AbhiPrasad.
|
To help us debug, is it possible you can set If you cannot turn this on no worries, we will still work toward reproducing/fixing this! Appreciate the patience. |
It's on our prod sorry, we'll stick with v7.49 for now. Thanks again for the quick response. |
We'll cut a patch release today that should hopefully fix this. Sorry for the inconvenience! |
Is there an existing issue for this?
How do you use Sentry?
Sentry Saas (sentry.io)
Which SDK are you using?
@sentry/node
SDK Version
7.51.2
Framework Version
No response
Link to Sentry event
https://algolia-site-search-crawler.sentry.io/issues/4155211403/?project=1260143
SDK Setup
Steps to Reproduce
Run our process as before
Expected Result
No unhandled error coming from the Undici integration
Actual Result
Those unhandled errors started to make our processes crash, immediately after we updated to Sentry v7.50, which enables the Undici integration by default (#7967):
The text was updated successfully, but these errors were encountered: