refactor: remove email validation logic from signInWithEmail and AuthService

Author: tomast1337

server/src/auth/auth.controller.ts

@@ -16,6 +16,7 @@ import type { Request, Response } from 'express';
 
 import { AuthService } from './auth.service';
 import { MagicLinkEmailStrategy } from './strategies/magicLinkEmail.strategy';
+import { Throttle } from '@nestjs/throttler';
 
 @Controller('auth')
 @ApiTags('auth')
@@ -51,7 +52,6 @@ export class AuthController {
     },
   })
   public async signInWithEmail(@Req() req: Request, @Res() res: Response) {
-    await this.authService.validateEmail(req.body);
     return this.magicLinkEmailStrategy.send(req, res);
   }
 

server/src/auth/auth.service.ts

@@ -261,32 +261,4 @@ export class AuthService {
 
     return user;
   }
-
-  public async validateEmail(body: Record<string, string | undefined>) {
-    // get destination from body
-    const { destination } = body;
-
-    if (!destination) {
-      throw new HttpException(
-        {
-          error: 'destination email not provided',
-        },
-        HttpStatus.BAD_REQUEST,
-      );
-    }
-
-    // find user by email
-    const user = await this.userService.findByEmail(destination);
-
-    if (!user) {
-      throw new HttpException(
-        {
-          error: 'User not found',
-        },
-        HttpStatus.NOT_FOUND,
-      );
-    } else {
-      return user;
-    }
-  }
 }