Skip to content

Conversation

@yu-so-young2
Copy link
Member

@yu-so-young2 yu-so-young2 commented Jan 1, 2025

📝 작업 내용

  • jwt 토큰의 만료기간을 코드 → .yml 파일에서 관리하도록 변경
    • dev / prod 환경의 accessToken 만료시간을 분리하여 관리하기 위함입니다.
  • 만료기간 값은 application-jwt-{profile}.yml 에 위치
    • 아직 dev, prod 값을 변경하지는 않았고, 확인해주시고 문제없다면 merge 전에 github Secret 업데이트하면 될 것 같습니다.

🔗 참고할만한 자료(선택)

💬 리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

@yu-so-young2 yu-so-young2 added the fix 기능 수정 label Jan 1, 2025
@yu-so-young2 yu-so-young2 requested a review from ssosee January 1, 2025 14:32
@yu-so-young2 yu-so-young2 self-assigned this Jan 1, 2025
Copy link
Member

@ssosee ssosee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

항상 감사합니당!

path: /
path: /

token:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

token으로 따로 분리하지 말고 jwt 안에 넣는 방식 어떤가요?!

jwt:
  expire:
     access: 1
     refresh: 2

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

아아 넘 좋습니다~~!! 👍👍

@yu-so-young2
Copy link
Member Author

QA 기간 후에 머지!

@yu-so-young2 yu-so-young2 merged commit cb54860 into develop Jan 21, 2025
1 check passed
@yu-so-young2 yu-so-young2 deleted the DP-455 branch January 21, 2025 03:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix 기능 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants