`Sentry.sentryHandleError` failed to normalize Remix request on Node v22

[clovis1122]

Is there an existing issue for this?

• I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
• I have reviewed the documentation https://docs.sentry.io/
• I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases

How do you use Sentry?

Sentry Saas (sentry.io)

Which SDK are you using?

@sentry/remix

SDK Version

8.9.1

Framework Version

Remix 2.9.2

Link to Sentry event

No response

SDK Setup

No response

Steps to Reproduce

Throw an error inside a loader function.

Expected Result

Sentry log the error with request information

Actual Result

Sentry shows "failed to normalize Remix request on Node". No error is logged (this bit I'm not sure why).

Adding some logs on Sentry, I see the following:

TypeError: Cannot read properties of undefined (reading 'depth')
    at [nodejs.util.inspect.custom] (node:internal/deps/undici/undici:3398:9)
    at Object.normalizeRemixRequest (/workspace/remix/node_modules/@sentry/remix/cjs/utils/web-fetch.js:132:63)
    at captureRemixServerException (/workspace/remix/node_modules/@sentry/remix/cjs/utils/instrumentServer.js:121:34)

The issue seems to be this code:

sentry-javascript/packages/remix/src/utils/web-fetch.ts

Line 150 in 679e149

headers: headers[Symbol.for('nodejs.util.inspect.custom')](),

This is all pretty similar to the issue I reported a while back. I feel like the current solution may break anytime the internals change. It doesn't have to. Node v22 will be LTS in 4 months, and Remix v2 already has a flag to use node's fetch implementation on the server instead of web fetch.

[Lms24]

Hey @clovis1122 thanks for writing in!

@onurtemizkan when you have a minute, would you mind taking a look? Any chance this will be fixed/no longer an issue by #12110?

[onurtemizkan]

Thanks for the issue report @clovis1122.

@Lms24, looks like #12479 will resolve this. I'll test and review today.

[kiliman]

Yes, my PR will fix this, but realized you're supporting older JS versions, so need to change from Object.fromEntries(). I'll update the PR and add a test case