Skip to content
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

Object.captureException #5425

Closed
3 tasks done
alexblack opened this issue Jul 15, 2022 · 1 comment · Fixed by #5441
Closed
3 tasks done

Object.captureException #5425

alexblack opened this issue Jul 15, 2022 · 1 comment · Fixed by #5441

Comments

@alexblack
Copy link

Is there an existing issue for this?

How do you use Sentry?

Sentry Saas (sentry.io)

Which package are you using?

@sentry/remix

SDK Version

7.7.0

Framework Version

7.7.0

Link to Sentry event

https://sentry.io/organizations/syncwith/issues/3429912339/?project=5880196&query=is%3Aunresolved&statsPeriod=24h

Steps to Reproduce

This started happening on our production site as soon as we switched to sentry-remix (previously we were using sentry-react and sentry-node separately)

Expected Result

I am not sure. If this represents an actual problem in our code, then I expected an appropriately titled issue to show up in Sentry, but its not a problem then I expected no issue.

Actual Result

An unexpected issue

Screen Shot 2022-07-15 at 11 39 15 AM

.
@alexblack
Copy link
Author

alexblack commented Jul 15, 2022

This seems to happen when our loader 404s:

export const loader: LoaderFunction = async ({ request, params }) => {
  throw new Response('not found', { status: 404 });
}

And I see in dev:

TypeError: body used already for: 
    at Response.consumeBody (/Users/alex/dev/micro/webapp/node_modules/node-fetch/lib/index.js:343:30)
    at Response.text (/Users/alex/dev/micro/webapp/node_modules/node-fetch/lib/index.js:283:22)
    at Object.extractData (/Users/alex/dev/micro/webapp/node_modules/@remix-run/server-runtime/data.js:138:19)
    at handleDocumentRequest (/Users/alex/dev/micro/webapp/node_modules/@remix-run/server-runtime/server.js:329:28)
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at requestHandler (/Users/alex/dev/micro/webapp/node_modules/@remix-run/server-runtime/server.js:49:18)
    at /Users/alex/dev/micro/webapp/node_modules/@sentry/remix/cjs/utils/instrumentServer.js:167:16
    at /Users/alex/dev/micro/webapp/node_modules/@remix-run/express/server.js:41:22

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants