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

[Feat] JWT Access Token 적용 #25 #27

Merged
merged 2 commits into from
May 12, 2023
Merged

[Feat] JWT Access Token 적용 #25 #27

merged 2 commits into from
May 12, 2023

Conversation

eunki96
Copy link
Member

@eunki96 eunki96 commented May 11, 2023

📋 이슈 번호

🛠 구현 사항

  • JWT Access Token 구현

📚 기타

  • JWT 액세스 토큰을 구현했습니다. 다만, 구현 과정에서 다소 무리하게 user.User 클래스에서 UserDetails를 implements 하여 부자연스러운 부분이 있습니다. 해당 부분 확인 후 문제가 없으면 merge 부탁드립니다.

📋 이슈 내용
JWT Refresh Token 기능을 적용합니다.

✅ 체크리스트
- user.User -> UserDetails implements 받음

📚 레퍼런스
@eunki96 eunki96 added the ✨ New Feature 신규 기능 (목표, 달성률) label May 11, 2023
@eunki96 eunki96 requested a review from Juser0 May 11, 2023 11:09
Copy link
Member

@Juser0 Juser0 left a comment

Choose a reason for hiding this comment

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

수고했어~
UserDetails 인터페이스를 상속하는 게 service류 method 하나 뿐이라 UserDetailsService 같은 식으로 구현하는 편이긴 한데, 사실 크게 상관 없다고 봐!
merge는 할까? 어떻게 할지 알려줘!

@eunki96
Copy link
Member Author

eunki96 commented May 11, 2023

오케이! merge 해'줘'

- Swagger UI 첫 페이지, 회원가입/프로필생성/로그인 페이지는 모두 접근 가능합니다.
- 그 외 페이지는 인가 받아야 접근 가능합니다.

- 해결해야 할 점 : UNAUTHORIZED_USER 일 때 CustomResponseDto 설정 한대로 반환되지 않아서 이 부분 고쳐야 합니다.
@Juser0 Juser0 merged commit 3c4cfc0 into main May 12, 2023
@Juser0 Juser0 deleted the eunki96/#25 branch May 12, 2023 07:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ New Feature 신규 기능 (목표, 달성률)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants