diff --git a/apps/scandic-web/app/[lang]/(partner)/(sas)/(protected)/sas-x-scandic/otp/OneTimePasswordForm.tsx b/apps/scandic-web/app/[lang]/(partner)/(sas)/(protected)/sas-x-scandic/otp/OneTimePasswordForm.tsx index 00c5b0164..cbe1353d7 100644 --- a/apps/scandic-web/app/[lang]/(partner)/(sas)/(protected)/sas-x-scandic/otp/OneTimePasswordForm.tsx +++ b/apps/scandic-web/app/[lang]/(partner)/(sas)/(protected)/sas-x-scandic/otp/OneTimePasswordForm.tsx @@ -73,10 +73,8 @@ export default function OneTimePasswordForm({ switch (requestOtp.data?.status) { case "ABUSED": - router.push(`/${params.lang}/sas-x-scandic/error?errorCode=tooManyCodes`) - return case "NOTSENT": - router.push(`/${params.lang}/sas-x-scandic/error`) + router.push(`/${params.lang}/sas-x-scandic/error?errorCode=tooManyCodes`) return case "NULL": case "RETRY": diff --git a/apps/scandic-web/server/routers/partners/sas/linkAccount.ts b/apps/scandic-web/server/routers/partners/sas/linkAccount.ts index 58eeb873f..cc760b3d7 100644 --- a/apps/scandic-web/server/routers/partners/sas/linkAccount.ts +++ b/apps/scandic-web/server/routers/partners/sas/linkAccount.ts @@ -1,3 +1,4 @@ +import * as Sentry from "@sentry/nextjs" import { z } from "zod" import * as api from "@/lib/api" @@ -59,9 +60,9 @@ export const linkAccount = protectedProcedure return { linkingState: "alreadyLinked" } } - console.log( - `[SAS] link account error with status code ${apiResponse.status} and response ${await apiResponse.text()}` - ) + const errorMessage = `[SAS] link account error with status code ${apiResponse.status} and response ${await apiResponse.text()}` + console.warn(errorMessage) + Sentry.captureMessage(errorMessage) return { linkingState: "error" } }) diff --git a/apps/scandic-web/server/routers/partners/sas/otp/request/requestOtp.ts b/apps/scandic-web/server/routers/partners/sas/otp/request/requestOtp.ts index a840fc3c9..a5accf2b2 100644 --- a/apps/scandic-web/server/routers/partners/sas/otp/request/requestOtp.ts +++ b/apps/scandic-web/server/routers/partners/sas/otp/request/requestOtp.ts @@ -1,3 +1,4 @@ +import * as Sentry from "@sentry/nextjs" import { TRPCError } from "@trpc/server" import { cookies } from "next/headers" import { v4 as uuidv4 } from "uuid" @@ -43,7 +44,6 @@ export const requestOtp = protectedProcedure const sasAuthToken = getSasToken() if (!sasAuthToken) { - // TODO: Should we verify that the SAS token isn't expired? throw createError("AUTH_TOKEN_NOT_FOUND") } @@ -67,6 +67,10 @@ export const requestOtp = protectedProcedure if (parseResult.data.status === "SENT") { setSASOtpCookie(parseResult.data) + } else { + const sasRequestOtpErrorMessage = `[SAS] requestOtp did not return SENT status with body: ${body}` + console.warn(sasRequestOtpErrorMessage) + Sentry.captureMessage(sasRequestOtpErrorMessage) } return parseResult.data diff --git a/apps/scandic-web/server/routers/partners/sas/performLevelUpgrade.ts b/apps/scandic-web/server/routers/partners/sas/performLevelUpgrade.ts index 92fb79153..fea75fb27 100644 --- a/apps/scandic-web/server/routers/partners/sas/performLevelUpgrade.ts +++ b/apps/scandic-web/server/routers/partners/sas/performLevelUpgrade.ts @@ -71,13 +71,13 @@ export const performLevelUpgrade = protectedProcedure if (notLinked) { const tierMatchErrorNotLinkedMessage = "[SAS] tier match error - not linked" - console.log(tierMatchErrorNotLinkedMessage) + console.warn(tierMatchErrorNotLinkedMessage) Sentry.captureMessage(tierMatchErrorNotLinkedMessage) return { tierMatchState: "notLinked" } } const tierMatchErrorMessage = `[SAS] tier match error with status code ${apiResponse.status} and response ${await apiResponse.text()}` - console.log(tierMatchErrorMessage) + console.error(tierMatchErrorMessage) Sentry.captureException(new Error(tierMatchErrorMessage)) return { tierMatchState: "error" } })