[FEAT] JWT 재발급 API - #54 #56

sjk4618 · 2024-07-10T15:43:06Z

🔥Pull requests

⛳️ 작업한 브랜치

feature/[FEAT] JWT 재발급 API #54

👷 작업한 내용

RefreshToken 재발급

@Transactional
    public UserJwtInfoRes reissue(final String refreshToken) {
        RefreshToken foundRefreshToken = getRefreshTokenByToken(refreshToken);
        jwtProvider.validateRefreshToken(foundRefreshToken.getExpiredAt());
        Long userId = foundRefreshToken.getUserId();
        Token newToken = jwtProvider.issueToken(userId);
        return UserJwtInfoRes.of(userId, newToken.accessToken(), newToken.refreshToken());
    }

토큰 만료기간 체크

    //refreshToken 재발급할 때 검증
    public void validateRefreshToken(LocalDateTime expireDate) {
        if (expireDate.isBefore(LocalDateTime.now())) {
            throw new UnauthorizedException(FailureCode.EXPIRED_REFRESH_TOKEN);
        }
    }

📟 관련 이슈

Resolved: [FEAT] JWT 재발급 API #54

yummygyudon

오랜만에 앱잼단 리뷰하니까 재밌네요!!
서버 전향하셔서 고생이 많으실텐데 항상 응원합니다! 잘하고 계세요! ❤️

yummygyudon · 2024-07-11T09:40:30Z

dateroad-api/src/main/java/org/dateroad/user/api/UserController.java

@sjk4618
패키징을 auth와 user 로 분리하신 것 같아요!
다만 조금 의아했던 것은
AuthService.java의 경우, user 패키지 내에 위치하고 있고
인증(auth) API는 UserController.java 안에 구현되어있는 것 같아요!!

혼재시켜놓은 이유가 있을까요?!

처음에 폴더링할 때, auth관련도 user에 다 넣으려고 했는데, 분리하는게 좋을 것 같습니다.
추후에 열려있는 브랜치들 머지하고 변경하도록 하겠습니다

yummygyudon · 2024-07-11T09:43:35Z

dateroad-api/src/main/java/org/dateroad/user/api/UserController.java

+    public ResponseEntity<UserJwtInfoRes> signUp(@RequestHeader(AUTHORIZATION) final String token,
+                                                 @RequestBody final UserSignUpReq userSignUPReq) {
+        UserJwtInfoRes userSignUpRes = authService.signUp(token, userSignUPReq);
        return ResponseEntity
                .status(HttpStatus.CREATED)
                .body(userSignUpRes);
    }

    @PostMapping("/signin")
-    public ResponseEntity<UserSignInRes> signIn(@RequestHeader(AUTHORIZATION) final String token,
+    public ResponseEntity<UserJwtInfoRes> signIn(@RequestHeader(AUTHORIZATION) final String token,


기존 Response DTO 이름이 UserSignInRes로 훨씬 직관적인 것 같아요!
DTO의 이름을 행위(기능)에 초점을 맞출 것인지, 내용에 초점 맞출 것인지 일관된 결정을 하면 더 좋을 것 같아요!

아 다시보니 로그인, 회원가입, jwt재발급에서 모두 같은 값을 주기때문에, 저렇게 하나로 사용했습니다

yummygyudon · 2024-07-11T09:45:35Z

dateroad-api/src/main/java/org/dateroad/user/service/AuthService.java

+            log.error(e.getMessage());
+            throw new UnauthorizedException(FailureCode.INVALID_REFRESH_TOKEN_VALUE);
+        } catch (Exception e) {
+            log.info(e.getMessage());


IllegalArgumentException 에 대해서는 log.error 를 찍으시구
Exception에 대해서는 log.info를 찍으신 이유가 따로 있을까요?!

아아 error로 바꾸겠습니다.

yummygyudon · 2024-07-11T09:48:57Z

dateroad-api/src/main/java/org/dateroad/user/service/AuthService.java

+            throw new UnauthorizedException(FailureCode.INVALID_REFRESH_TOKEN_VALUE);
+        } catch (Exception e) {
+            log.info(e.getMessage());
+            throw new RuntimeException("An unexpected error occurred", e);


어플리케이션 Custom Exception 클래스를 구현했고 해당 Custom Exception에서도 RuntimeException 상속받은 것 같아요!
그렇기 때문에 굳이 RuntimeException 객체를 throw 한 이유가 궁금합니다!!

그리고 Message 값은 매직 String(private static String) 변수로 선언하고 주입시켜주면 훨씬 직관적일 것 같습니다!

넵! 저희가 커스텀한 exception으로 throw 하겠습니다

# Conflicts: # dateroad-api/src/main/java/org/dateroad/user/api/UserController.java # dateroad-api/src/main/java/org/dateroad/user/service/AuthService.java # dateroad-external/src/main/java/org/dateroad/feign/kakao/KakaoPlatformUserIdProvider.java

rlarlgnszx

수고하셨습니다!

sjk4618 added 5 commits July 11, 2024 00:23

[feat] RefreshTokenRepository 메서드 추가 - #54

b12d28c

[feat] 리프레시토큰 저장방식 변경 - #54

e50ec2e

[feat] 재발급할 때, 만료 체크 - #54

9a61212

[feat] AuthService에 재발급 메서드 추가 - #54

850c254

[chore] 코드 정리 - #54

0764fa5

sjk4618 added feat 성준 labels Jul 10, 2024

sjk4618 requested review from rlarlgnszx and gardening-y July 10, 2024 15:43

sjk4618 self-assigned this Jul 10, 2024

sjk4618 added 3 commits July 11, 2024 00:55

[chore] final 추가 - #54

64f924a

[feat] 재로그인시 refreshToken 삭제 - #54

0490ae8

[chore] 코드 정리 - #54

75a0d5d

yummygyudon reviewed Jul 11, 2024

View reviewed changes

sjk4618 added 2 commits July 11, 2024 22:18

[chore] 코드리뷰 반영 - #54

a815ac0

[merge] conflict 해결 - #54

cdcd5e8

# Conflicts: # dateroad-api/src/main/java/org/dateroad/user/api/UserController.java # dateroad-api/src/main/java/org/dateroad/user/service/AuthService.java # dateroad-external/src/main/java/org/dateroad/feign/kakao/KakaoPlatformUserIdProvider.java

rlarlgnszx approved these changes Jul 11, 2024

View reviewed changes

[chore] 코드 정리 - #54

922b219

sjk4618 merged commit c5d46bd into develop Jul 11, 2024
1 check passed

sjk4618 deleted the feature/#54 branch July 11, 2024 19:05

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[FEAT] JWT 재발급 API - #54 #56

[FEAT] JWT 재발급 API - #54 #56

sjk4618 commented Jul 10, 2024

yummygyudon left a comment

yummygyudon Jul 11, 2024

sjk4618 Jul 11, 2024

yummygyudon Jul 11, 2024

sjk4618 Jul 11, 2024

sjk4618 Jul 11, 2024

yummygyudon Jul 11, 2024

sjk4618 Jul 11, 2024

yummygyudon Jul 11, 2024

sjk4618 Jul 11, 2024

rlarlgnszx left a comment

[FEAT] JWT 재발급 API - #54 #56

[FEAT] JWT 재발급 API - #54 #56

Conversation

sjk4618 commented Jul 10, 2024

🔥Pull requests

📟 관련 이슈

yummygyudon left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

rlarlgnszx left a comment

Choose a reason for hiding this comment