Refactor: Add error case

src/auth/auth.controller.ts

@@ -6,6 +6,7 @@ import { Response } from 'express';
 
 import { KakaoInfo } from 'src/common/decorators/kakao-info.decorator';
 import { KakaoGuard } from 'src/common/guards/kakao.guard';
+import { getCookieOption } from 'src/common/helper/cookie.helper';
 import { UserProvider } from 'src/entities';
 
 import { AuthService } from './auth.service';
@@ -32,14 +33,11 @@ export class AuthController {
       kakaoRefreshToken: refreshToken,
     });
 
-    res.cookie('Authorization', 'Bearer ' + user.accessToken, {
-      httpOnly: true,
-      sameSite: 'none',
-      secure: true,
-      path: '/',
-      expires: new Date(new Date().getTime() + 1000 * 60 * 60 * 24 * 1 * 365),
-      domain: '.korrk.kr',
-    });
+    res.cookie(
+      'Authorization',
+      'Bearer ' + user.accessToken,
+      getCookieOption(),
+    );
 
     return res.redirect(302, this.configService.get('CLIENT_URL'));
   }

src/auth/strategies/jwt.strategy.ts

@@ -1,9 +1,14 @@
-import { Injectable, UnauthorizedException } from '@nestjs/common';
+import {
+  Injectable,
+  NotFoundException,
+  UnauthorizedException,
+} from '@nestjs/common';
 import { ConfigService } from '@nestjs/config';
 import { PassportStrategy } from '@nestjs/passport';
 
 import { ExtractJwt, Strategy, VerifiedCallback } from 'passport-jwt';
 
+import { User } from 'src/entities/index';
 import { UserService } from 'src/user/user.service';
 
 @Injectable()
@@ -19,12 +24,14 @@ export class JwtStrategy extends PassportStrategy(Strategy, 'jwt') {
   }
 
   async validate(payload: { id: number }, done: VerifiedCallback) {
-    const user = await this.userService.findOne({
+    const user: User = await this.userService.findOne({
       id: payload.id,
     });
     if (!user) {
       throw new UnauthorizedException('존재하지 않는 유저입니다.');
     }
+    if (user.nickname == null)
+      throw new NotFoundException('닉네임 설정을 완료해주세요.');
     done(null, user);
   }
 }

src/common/helper/cookie.helper.ts

@@ -0,0 +1,21 @@
+import { CookieOptions } from 'express';
+
+import { getNodeEnv } from 'src/common/helper/env.helper';
+import { NODE_ENVIRONMENT } from 'src/common/helper/env.validation';
+
+export const getCookieOption = (): CookieOptions => {
+  if (getNodeEnv === NODE_ENVIRONMENT.development) {
+    return {
+      path: '/',
+      maxAge: 360000,
+    };
+  }
+  return {
+    httpOnly: true,
+    sameSite: 'none',
+    secure: true,
+    path: '/',
+    expires: new Date(new Date().getTime() + 1000 * 60 * 60 * 24 * 1 * 365),
+    domain: '.korrk.kr',
+  };
+};