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

[#66] Jwt & OAuth 리팩터링 및 테스트 추가 #74

Merged
merged 9 commits into from
Jan 25, 2023

Conversation

JoosungKwon
Copy link
Member

🌱 작업 사항

JWT

  • headerkey 변경 (token -> AUTHORIZATION Bearer 로 변경)
  • payload(Claim) 변경 (email -> id)
    • email은 중복의 가능성이 있기 때문에 확장성 고려를 위해 변경하는 것이 좋습니다.
  • 유효시간 1분 -> 20분 변경

OAuth

  • ProfileImgUrl 추가
    • 카카오에서 프로필 이미지도 받아오도록 설정
    • 엔티티에 필드 추가 및 flyway V2추가

테스트

  • JwtConfig 추가 (secretkey 변수화)

🦄 관련 이슈

#66

- 이메일을 사용하고 있는 controller 및 service
- Oauth에서 프로필 이미지 URL도 가져오도록 함
- UserProfile 및 UserInfo 변경 -> 네이밍 치환
- Token 유효 시간 1분 -> 20분으로 변경
- JwtConfig로 secretKey 변수화
- 기존 코드 변경사항 반영
@JoosungKwon JoosungKwon added ♻️ refactor 코드 리팩터링 ✅ test 테스트 추가/수정 labels Jan 24, 2023
@JoosungKwon JoosungKwon self-assigned this Jan 24, 2023
- profile_img_url 필드 추가
- profile_img_url 필드 추가
@JoosungKwon JoosungKwon merged commit 7870753 into develop Jan 25, 2023
@JoosungKwon JoosungKwon deleted the refactor/66-change-jwt-payload branch January 25, 2023 05:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ refactor 코드 리팩터링 ✅ test 테스트 추가/수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants