Error handling

[niwsa]

I have been using this strategy to implement remix-auth-saml. Couple of questions I have

1. Is it intentional to throw 401 here rather than catching it with the this.failure method like it's done for verify ?

   remix-auth-oauth2/src/index.ts

   Line 331 in 7857211

   throw new Response(body, { status: 401 });

2. Should the callback routes (e.g. app/routes/auth/github/callback.tsx) export a default component so that it's treated as a non-resource route and any error gets caught in the app root CatchBoundary ? Unless I put something like below the CatchBoundary is not shown.

export default function Render() {
  return null;
}

[sergiodxa]

That's not correct, it should use this.failure instead.