From cf19bc70541544bedc610865156b8faa1eb377fa Mon Sep 17 00:00:00 2001 From: Kalle <38327916+Sendouc@users.noreply.github.com> Date: Tue, 26 Sep 2023 23:58:31 +0300 Subject: [PATCH] Add error log when finishing auth fails --- app/modules/auth/DiscordStrategy.server.ts | 33 +++++++++++++--------- 1 file changed, 19 insertions(+), 14 deletions(-) diff --git a/app/modules/auth/DiscordStrategy.server.ts b/app/modules/auth/DiscordStrategy.server.ts index 2445bc7d2..a5d7cb29d 100644 --- a/app/modules/auth/DiscordStrategy.server.ts +++ b/app/modules/auth/DiscordStrategy.server.ts @@ -55,23 +55,28 @@ export class DiscordStrategy extends OAuth2Strategy< callbackURL: new URL("/auth/callback", envVars.BASE_URL).toString(), }, async ({ accessToken }) => { - const discordResponses = this.authGatewayEnabled() - ? await this.fetchProfileViaGateway(accessToken) - : await this.fetchProfileViaDiscordApi(accessToken); + try { + const discordResponses = this.authGatewayEnabled() + ? await this.fetchProfileViaGateway(accessToken) + : await this.fetchProfileViaDiscordApi(accessToken); - const [user, connections] = - discordUserDetailsSchema.parse(discordResponses); + const [user, connections] = + discordUserDetailsSchema.parse(discordResponses); - const userFromDb = db.users.upsert({ - discordAvatar: user.avatar ?? null, - discordDiscriminator: user.discriminator, - discordId: user.id, - discordName: user.global_name ?? user.username, - discordUniqueName: user.global_name ? user.username : null, - ...this.parseConnections(connections), - }); + const userFromDb = db.users.upsert({ + discordAvatar: user.avatar ?? null, + discordDiscriminator: user.discriminator, + discordId: user.id, + discordName: user.global_name ?? user.username, + discordUniqueName: user.global_name ? user.username : null, + ...this.parseConnections(connections), + }); - return userFromDb.id; + return userFromDb.id; + } catch (e) { + console.error("Failed to finish authentication:\n", e); + throw new Error("Failed to finish authentication"); + } }, );