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

Handling TRPC errors in Sentry #14004

Open
MonkeeMan1 opened this issue Oct 16, 2024 · 4 comments
Open

Handling TRPC errors in Sentry #14004

MonkeeMan1 opened this issue Oct 16, 2024 · 4 comments

Comments

@MonkeeMan1
Copy link

Environment

SaaS (https://sentry.io/)

What are you trying to accomplish?

Hello,

I have recently implemented Sentry into my webapp that uses TRPC. However, we are running into problems with Sentry displaying error pages to our users for client side errors which do not break functionality. See image below
Image

How are you getting stuck?

An example of when this page appears is when a user tries to visit a page which requires them to be signed in.
On our protectedProcedure function we check if the user is signed in, and if they aren't we throw a TRPC unauthorized error.

export const protectedProcedure = t.procedure.use(({ ctx, next }) => {
	if (!ctx.session || !ctx.session.user) {
		throw new TRPCError({ code: 'UNAUTHORIZED', message: 'You are not signed in' });
	}

This is resulting in the error below: Image

We are using an error boundary provided by sentry, with no props: import { ErrorBoundary } from '@sentry/nextjs'; and wrapping that inside of our root layout.

How would I go about stopping sentry from detecting these TRPC errors?

Where in the product are you?

Unknown

Link

No response

DSN

No response

Version

No response

@getsantry
Copy link

getsantry bot commented Oct 16, 2024

Assigning to @getsentry/support for routing ⏲️

@getsantry
Copy link

getsantry bot commented Oct 16, 2024

Routing to @getsentry/product-owners-issues for triage ⏲️

@jangjodi
Copy link
Member

Hi @ReneGreen27, this seems to be an SDK issue - could we please re-route this?

@ReneGreen27 ReneGreen27 transferred this issue from getsentry/sentry Oct 16, 2024
@chargome
Copy link
Member

Hi @MonkeeMan1, the trpc middleware currently does not support filtering your errors directly but you can do so following these docs.

As for your client, simply catching and handling your errors should prevent them from hitting the error boudary.

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

No branches or pull requests

3 participants