better check for existing emails in the profile info

Author: ericallam

apps/webapp/app/models/user.server.ts

@@ -11,17 +11,6 @@ export type { User } from "@trigger.dev/database";
 import { assertEmailAllowed } from "~/utils/email";
 import { logger } from "~/services/logger.server";
 
-export class AuthConflictError extends Error {
-  constructor(
-    message: string,
-    public readonly existingEmailUserId: string,
-    public readonly existingAuthUserId: string
-  ) {
-    super(message);
-    this.name = "AuthConflictError";
-  }
-}
-
 type FindOrCreateMagicLink = {
   authenticationMethod: "MAGIC_LINK";
   email: string;

apps/webapp/app/services/gitHubAuth.server.ts

@@ -20,7 +20,7 @@ export function addGitHubStrategy(
     async ({ extraParams, profile }) => {
       const emails = profile.emails;
 
-      if (!emails) {
+      if (!emails?.length) {
         throw new Error("GitHub login requires an email address");
       }
 

apps/webapp/app/services/googleAuth.server.ts

@@ -20,7 +20,7 @@ export function addGoogleStrategy(
     async ({ extraParams, profile }) => {
       const emails = profile.emails;
 
-      if (!emails) {
+      if (!emails?.length) {
         throw new Error("Google login requires an email address");
       }
 
@@ -52,4 +52,3 @@ export function addGoogleStrategy(
 
   authenticator.use(googleStrategy);
 }
-