fix: don't add /error to url pathname when email verification is successful (#6492)

fix handleAuthorized making bad pth when authorize

Co-authored-by: Corey Jepperson <[email protected]>
Co-authored-by: Thang Vu <[email protected]>

Author: 3 people

packages/core/src/lib/routes/shared.ts

@@ -7,15 +7,16 @@ export async function handleAuthorized(
   params: any,
   { url, logger, callbacks: { signIn } }: InternalOptions
 ) {
-  url.pathname += "/error"
   try {
     const authorized = await signIn(params)
     if (!authorized) {
+      url.pathname += "/error"
       logger.debug("User not authorized", params)
       url.searchParams.set("error", "AccessDenied")
       return { status: 403 as const, redirect: url.toString() }
     }
   } catch (e) {
+    url.pathname += "/error"
     const error = new AuthorizedCallbackError(e as Error)
     logger.error(error)
     url.searchParams.set("error", "Configuration")