🐛 fix: OAuth 서버 로그인 중 실패에 대한 리디렉션 추가

server/src/user/controller/oAuth.controller.ts

@@ -38,7 +38,7 @@ export class OAuthController {
     @Res() res: Response,
   ) {
     await this.oauthService.callback(callbackDto, res);
-    return res.redirect(`${OAUTH_URL_PATH.BASE_URL}/oauth-success`);
+    return res;
   }
 
   @Get('e2e/callback')

server/src/user/dto/request/oAuthCallbackDto.ts

@@ -1,19 +1,20 @@
-import { ApiProperty } from '@nestjs/swagger';
+import { ApiProperty, ApiPropertyOptional } from '@nestjs/swagger';
 
-import { IsNotEmpty, IsString } from 'class-validator';
+import { IsNotEmpty, IsOptional, IsString } from 'class-validator';
 
 export class OAuthCallbackRequestDto {
-  @ApiProperty({
+  @ApiPropertyOptional({
     example: 'testCode',
     description: 'Access Token 갱신 토큰',
   })
+  @IsOptional()
   @IsNotEmpty({
-    message: 'code는 필수 입력 값입니다.',
+    message: 'code는 빈 문자열일 수 없습니다.',
   })
   @IsString({
     message: '문자열로 입력해주세요.',
   })
-  code: string;
+  code?: string;
 
   @ApiProperty({
     example: '{ provider: {플랫폼 종류}}',
@@ -27,6 +28,19 @@ export class OAuthCallbackRequestDto {
   })
   state: string;
 
+  @ApiPropertyOptional({
+    example: 'error=access_denied',
+    description: 'OAuth 서버 로그인 중 실패 발생 시 반환되는 값',
+  })
+  @IsOptional()
+  @IsNotEmpty({
+    message: 'error는 빈 문자열일 수 없습니다.',
+  })
+  @IsString({
+    message: '문자열로 입력해주세요.',
+  })
+  error?: string;
+
   constructor(partial: Partial<OAuthCallbackRequestDto>) {
     Object.assign(this, partial);
   }

server/src/user/service/oAuth.service.ts

@@ -12,7 +12,11 @@ import { cookieConfig } from '@common/cookie/cookie.config';
 import { Payload } from '@common/guard/jwt.guard';
 import { WinstonLoggerService } from '@common/logger/logger.service';
 
-import { OAuthType, StateData } from '@user/constant/oauth.constant';
+import {
+  OAUTH_URL_PATH,
+  OAuthType,
+  StateData,
+} from '@user/constant/oauth.constant';
 import { REFRESH_TOKEN_TTL } from '@user/constant/user.constants';
 import { OAuthCallbackRequestDto } from '@user/dto/request/oAuthCallbackDto';
 import { Provider } from '@user/entity/provider.entity';
@@ -43,6 +47,10 @@ export class OAuthService {
     const stateData = this.parseStateData(callbackDto.state);
     const { provider: providerType } = stateData;
 
+    if (callbackDto.error || !callbackDto.code) {
+      return res.redirect(`${OAUTH_URL_PATH.BASE_URL}/signin`);
+    }
+
     const tokenData = await this.providers[providerType].getTokens(
       callbackDto.code,
     );
@@ -60,6 +68,8 @@ export class OAuthService {
       },
       res,
     );
+
+    return res.redirect(`${OAUTH_URL_PATH.BASE_URL}/oauth-success`);
   }
 
   async e2eCallback(providerType: OAuthType, res: Response) {

server/test/oauth/dto/oauth-callback.dto.spec.ts

@@ -23,18 +23,6 @@ describe('OAuthCallbackRequestDto Test', () => {
   });
 
   describe('code', () => {
-    it('code가 없을 경우 유효성 검사에 실패한다.', async () => {
-      // given
-      dto.code = null;
-
-      // when
-      const errors = await validate(dto);
-
-      // then
-      expect(errors).toHaveLength(1);
-      expect(errors[0].constraints).toHaveProperty('isNotEmpty');
-    });
-
     it('code가 빈 문자열일 경우 유효성 검사에 실패한다.', async () => {
       // given
       dto.code = '';
@@ -89,10 +77,47 @@ describe('OAuthCallbackRequestDto Test', () => {
       // given
       dto.state = 1 as any;
 
-      // when
+      //when
       const errors = await validate(dto);
 
-      // then
+      //then
+      expect(errors).toHaveLength(1);
+      expect(errors[0].constraints).toHaveProperty('isString');
+    });
+  });
+
+  describe('error', () => {
+    it('error가 있을 경우 유효성 검사에 성공한다.', async () => {
+      //given
+      dto.error = 'access_denied';
+
+      //when
+      const errors = await validate(dto);
+
+      //then
+      expect(errors).toHaveLength(0);
+    });
+
+    it('error가 빈 문자열일 경우 유효성 검사에 실패한다.', async () => {
+      //given
+      dto.error = '';
+
+      //when
+      const errors = await validate(dto);
+
+      //then
+      expect(errors).toHaveLength(1);
+      expect(errors[0].constraints).toHaveProperty('isNotEmpty');
+    });
+
+    it('error가 문자가 아닌 숫자일 경우 유효성 검사에 실패한다.', async () => {
+      //given
+      dto.error = 1 as any;
+
+      //when
+      const errors = await validate(dto);
+
+      //then
       expect(errors).toHaveLength(1);
       expect(errors[0].constraints).toHaveProperty('isString');
     });