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

fix: 소셜로그인이 개발 환경, 배포 환경 모두에서 가능하도록 수정 #146

Merged
merged 3 commits into from
Aug 3, 2024

Conversation

ywonchae1
Copy link
Member

🌱 관련 이슈

📌 작업 내용 및 특이사항

  • 현재는 소셜로그인이 개발 환경 (localhost)에서만 가능한 상태입니다.
  • 배포 환경에서도 정상 동작할 수 있도록 HttpServletRequest의 Referer Header을 활용하여 동적인 redirect가 가능하도록 설정합니다.

📝 참고사항

  • 카카오와 구글 환경변수가 추가되었습니다. 해당 내용은 슬랙 캔버스에서 확인할 수 있습니다.

  • KAKAO_REDIRECT_URL은 서버 자체 내부에서 로그인을 진행할 때 사용하는 url입니다.

  • KAKAO_CLIENT_REDIRECT_URL은 클라이언트에서 로그인을 진행할 때 사용하는 url입니다.

  • 구글도 동일합니다.

  • 구글 소셜로그인 성공

Screenshot 2024-08-03 at 00 45 43
  • 카카오 소셜로그인 성공
Screenshot 2024-08-03 at 00 45 05
  • 데이터베이스
Screenshot 2024-08-03 at 00 50 33

@ywonchae1 ywonchae1 added the fix label Aug 2, 2024
@ywonchae1 ywonchae1 self-assigned this Aug 2, 2024
Copy link

github-actions bot commented Aug 2, 2024

Test Results

45 tests   45 ✅  2s ⏱️
11 suites   0 💤
11 files     0 ❌

Results for commit ecf459a.

Copy link
Collaborator

@penrose15 penrose15 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

@ywonchae1 ywonchae1 merged commit aa4ffb6 into develop Aug 3, 2024
2 checks passed
@ywonchae1 ywonchae1 deleted the fix/134-login-redirect-url branch August 3, 2024 03:34
@ywonchae1 ywonchae1 restored the fix/134-login-redirect-url branch August 3, 2024 05:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🐛 소셜로그인 개발, 배포 환경 모두에서 가능하도록 수정
2 participants