Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[COZY-438] fix : 토큰 만료시간 분리 및 관리자 토큰 발급 관련 오류 수정 #211

Merged
merged 1 commit into from
Dec 4, 2024

Conversation

genius00hwan
Copy link
Contributor

⚒️develop의 최신 커밋을 pull 받았나요?

#️⃣ 작업 내용

어떤 기능을 구현했나요?
기존 기능에서 어떤 점이 달라졌나요?
자세한 로직이 필요하다면 함께 적어주세요!
코드에 대한 설명이라면, 코맨트를 통해서 어떤 부분이 어떤 코드인지 설명해주세요!

동작 확인

기능을 실행했을 때 정상 동작하는지 여부를 확인하고 스샷을 올려주세요

💬 리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요
고민사항도 적어주세요.

@genius00hwan genius00hwan merged commit 6aafa35 into develop Dec 4, 2024
1 check passed
@genius00hwan genius00hwan deleted the hotfix/COZY-438 branch December 4, 2024 13:03
Copy link

github-actions bot commented Dec 4, 2024

리뷰해드려요~

JwtUtil.java - 리뷰

  1. 파일 내용 줄 수 증가: -1
  2. 멤버 변수 MEMBER_KEY 제거
  3. 멤버 변수 ACCESS_EXPIRATION 정의 변경
  • 주석 제거
  • 변경 속성 변경 (@Value("${jwt.access-token.expire-length}") 추가)

제안 개선 사항:

  1. 파일 내용 줄 수 증가: 1
  2. 멤버 변수 MEMBER_KEY 추가
  • 속성 변경 (@Value("${jwt.member-token.key}"))

이러한 변경으로 인해 회원 토큰과 관련된 코드가 추가되어야 할 수 있습니다.

SwaggerFilter.java - 리뷰

  1. 파일 내용 줄 수 증가: 0
  2. 멤버 변수 JWT_ADMIN_TOKEN 제거
  3. 코드 내용 줄 수 증가: 1
  • JWT_ADMIN_TOKEN 제거
  • jwtUtil.generateAdminToken() 사용

제안 개선 사항:

  1. 파일 내용 줄 수 증가: 0
  2. 코드 내용 줄 수 증가: 0

이러한 변경으로 인해 코드의 일관성과 가독성이 향상되었습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant