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

hotfix: logout 요청 시 토큰 만료로 인한 비즈니스 로직 실패 케이스 수정 #259

Merged
merged 5 commits into from
Sep 26, 2023

Conversation

YongsHub
Copy link
Collaborator

Issues

구분

  • 버그 수정
  • 기능 추가
  • 코드 리팩터링
  • 문서 업데이트
  • 기타

주요 변경점

  • accessToken parseClaims 과정에서 발생할 수 있는 예외 케이스를 try - catch를 통해 잡았습니다.
  • 어떠한 케이스에도 로그아웃 요청 시, Cookie Setting을 비울 수 있도록 코드를 리팩토링 했습니다.

스크린샷

기타

try - catch 리팩토링 코드 보시면서 의견 주신다면 감사합니다.

@YongsHub YongsHub added fix 버그 수정 refactor 리팩터링 hotfix labels Sep 25, 2023
@YongsHub YongsHub self-assigned this Sep 25, 2023
Copy link
Member

@jjuny0310 jjuny0310 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다

Copy link
Collaborator

@khs960616 khs960616 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다.

@moonn6pence
Copy link
Collaborator

@YongsHub YongsHub merged commit aa75402 into SSAF-SOUND:dev Sep 26, 2023
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix 버그 수정 hotfix refactor 리팩터링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants