Merge pull request #3 from TLX-Protocol/fail-silently-for-discord

Fail Silently for Discord Issues

functions/src/registration.ts

@@ -21,29 +21,34 @@ async function getTwitterUsername(code: string, secrets: Secrets): Promise<strin
   }
 }
 
-async function getDiscordData(code: string, secrets: Secrets): Promise<{ username: string; guilds: Guild[] }> {
+async function getDiscordData(
+  code: string,
+  secrets: Secrets
+): Promise<{ username: string; guilds: Guild[]; error: boolean }> {
   try {
     logger.info("Getting Discord data");
     logger.info(`Code: ${code}`);
     const discordService = await DiscordService.fromCode(code, secrets);
     const username = await discordService.getUsername();
     const guilds = await discordService.getGuilds();
-    return { username, guilds };
+    return { username, guilds, error: false };
   } catch (error) {
-    throw new APIError(`Discord authentication failed: ${error}`);
+    logger.error(`Discord authentication failed: ${error}`);
+    return { username: "ERROR", guilds: [], error: true };
   }
 }
 
 export default async function registrationHandler(request: Request, secrets: Secrets) {
   const params = validateParams<RegistrationParams>(request.body, ...requiredKeys);
   const address = getUserAddress(params.signature);
-
   const db = admin.database();
 
+  // Validating address
   if (await userExists(address)) {
     throw new APIError("Address already used");
   }
 
+  // Validating invite code
   const codeSnapshot = await db.ref("invites").child(params.inviteCode).get();
   if (!codeSnapshot.exists()) {
     throw new APIError("Invalid invite code");
@@ -52,25 +57,31 @@ export default async function registrationHandler(request: Request, secrets: Sec
     throw new APIError("Code already used");
   }
 
+  // Validating Twitter
   const twitterUsername = await getTwitterUsername(params.twitterCode, secrets);
-
   if (await usernameExists("twitterUsername", twitterUsername)) {
     throw new APIError("Twitter already used");
   }
 
-  const { username: discordUsername, guilds } = await getDiscordData(params.discordCode, secrets);
-
-  if (await usernameExists("discordUsername", discordUsername)) {
-    throw new APIError("Discord already used");
-  }
-
-  if (!guilds.some((guild) => guild.id === tlxGuidID)) {
-    throw new APIError("Not in the TLX Discord server");
+  // Validating Discord
+  const { username: discordUsername, guilds, error } = await getDiscordData(params.discordCode, secrets);
+  if (!error) {
+    if (await usernameExists("discordUsername", discordUsername)) {
+      throw new APIError("Discord already used");
+    }
+    if (!guilds.some((guild) => guild.id === tlxGuidID)) {
+      throw new APIError("Not in the TLX Discord server");
+    }
   }
 
+  // Saving user
   const user = { twitterUsername, discordUsername };
   await db.ref("users").child(address).set(user);
+
+  // Generating invite codes
   const codes = await generateAndSaveInviteCodes(address, invitesPerUser);
+
+  // Using invite code
   await useCode(params.inviteCode, address);
 
   return { address, ...user, codes };