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: support kakao oauth login #42

Merged
merged 7 commits into from
Feb 3, 2023

Conversation

junghoon-vans
Copy link
Member

@junghoon-vans junghoon-vans commented Feb 2, 2023

Related Issue

#27

Description

kakao 간편 로그인 기능을 추가하였습니다.
로컬 상에서 사용하는 URL은 다음과 같습니다.

  • 요청 URL: localhost:8080/oauth2/authorization/kakao
  • 리다이렉트 URL: http://localhost:8080/login/oauth2/code/kakao

Screenshots (if appropriate):

스크린샷 2023-02-02 오후 9 39 18

주의할 점은 모든 권한을 수락하지 않은면 데이터베이스 상에서 null 에러 발생합니다.
추후 위 이미지 상의 권한은 모두 필수로 지정해야 될 것 같습니다.

@junghoon-vans junghoon-vans added the ✨ feature 기능 추가가 필요한 경우를 명시합니다. label Feb 2, 2023
@junghoon-vans junghoon-vans self-assigned this Feb 2, 2023
@junghoon-vans junghoon-vans force-pushed the feature/support-kakao-oauth-login branch from 34b230d to 41a52ed Compare February 3, 2023 11:21
@sonarcloud
Copy link

sonarcloud bot commented Feb 3, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

64.3% 64.3% Coverage
0.0% 0.0% Duplication

Copy link
Member

@Chaerim1001 Chaerim1001 left a comment

Choose a reason for hiding this comment

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

테스트 코드까지 수고 많으셨습니다 👍

@junghoon-vans junghoon-vans merged commit 3b66a66 into develop Feb 3, 2023
@junghoon-vans junghoon-vans deleted the feature/support-kakao-oauth-login branch February 3, 2023 13:18
@junghoon-vans junghoon-vans changed the title feature: support kakao oauth login feat: support kakao oauth login Feb 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ feature 기능 추가가 필요한 경우를 명시합니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants