Removes unused try {} catch {}

[simonbs]

This PR removes a try {} catch {} on the sign in page. The catch block merely printed the error. However, this resulted in the following error being logged upon every sign in with GitHub.

Error: NEXT_REDIRECT
    at getRedirectError (webpack-internal:///(rsc)/./node_modules/next/dist/client/components/redirect.js:49:19)
    at redirect (webpack-internal:///(rsc)/./node_modules/next/dist/client/components/redirect.js:60:11)
    at signIn (webpack-internal:///(rsc)/./node_modules/next-auth/lib/actions.js:59:76)
    at async $$ACTION_0 (webpack-internal:///(rsc)/./src/app/auth/signin/page.tsx:252:9)
    at async /Users/simonbs/Developer/shape-docs/node_modules/next/dist/compiled/next-server/app-page.runtime.dev.js:39:418
    at async rw (/Users/simonbs/Developer/shape-docs/node_modules/next/dist/compiled/next-server/app-page.runtime.dev.js:38:7978)
    at async r6 (/Users/simonbs/Developer/shape-docs/node_modules/next/dist/compiled/next-server/app-page.runtime.dev.js:41:1256)
    at async doRender (/Users/simonbs/Developer/shape-docs/node_modules/next/dist/server/base-server.js:1438:30)
    at async cacheEntry.responseCache.get.routeKind (/Users/simonbs/Developer/shape-docs/node_modules/next/dist/server/base-server.js:1599:28)
    at async DevServer.renderToResponseWithComponentsImpl (/Users/simonbs/Developer/shape-docs/node_modules/next/dist/server/base-server.js:1507:28)
    at async DevServer.renderPageComponent (/Users/simonbs/Developer/shape-docs/node_modules/next/dist/server/base-server.js:1931:24)
    at async DevServer.renderToResponseImpl (/Users/simonbs/Developer/shape-docs/node_modules/next/dist/server/base-server.js:1969:32)
    at async DevServer.pipeImpl (/Users/simonbs/Developer/shape-docs/node_modules/next/dist/server/base-server.js:920:25)
    at async NextNodeServer.handleCatchallRenderRequest (/Users/simonbs/Developer/shape-docs/node_modules/next/dist/server/next-server.js:272:17)
    at async DevServer.handleRequestImpl (/Users/simonbs/Developer/shape-docs/node_modules/next/dist/server/base-server.js:816:17)
    at async /Users/simonbs/Developer/shape-docs/node_modules/next/dist/server/dev/next-dev-server.js:339:20
    at async Span.traceAsyncFn (/Users/simonbs/Developer/shape-docs/node_modules/next/dist/trace/trace.js:154:20)
    at async DevServer.handleRequest (/Users/simonbs/Developer/shape-docs/node_modules/next/dist/server/dev/next-dev-server.js:336:24)
    at async invokeRender (/Users/simonbs/Developer/shape-docs/node_modules/next/dist/server/lib/router-server.js:174:21)
    at async handleRequest (/Users/simonbs/Developer/shape-docs/node_modules/next/dist/server/lib/router-server.js:353:24)
    at async requestHandlerImpl (/Users/simonbs/Developer/shape-docs/node_modules/next/dist/server/lib/router-server.js:377:13)
    at async Server.requestListener (/Users/simonbs/Developer/shape-docs/node_modules/next/dist/server/lib/start-server.js:141:13) {
  digest: 'NEXT_REDIRECT;replace;https://github.com/login/oauth/authorize?scope=repo&response_type=code&client_id=...;',
  mutableCookies: p {
    _parsed: Map(3) {
      'authjs.pkce.code_verifier' => [Object],
      'authjs.callback-url' => [Object],
      'authjs.csrf-token' => [Object]
    },
    _headers: _HeadersList {
      cookies: [Array],
      [Symbol(headers map)]: [Map],
      [Symbol(headers map sorted)]: null
    }

As far as I can tell from this blog post this error isn't a "real" error that's meant to be handled but rather a signal to Next.js that it needs to do a redirect.

Since we're not doing any interesting error handling, I thought we might as well remove the try/catch entirely and rely on Next.js to handle any error thrown. At least this avoids polluting our logs with errors that we cannot and should not do anything about.

[ulrikandersen]

Makes sense 👍