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] 이벤트유저 로그인 기능 구현 (#17) #22

Merged
merged 13 commits into from
Jul 31, 2024

Conversation

win-luck
Copy link
Collaborator

@win-luck win-luck commented Jul 31, 2024

#️⃣ 연관 이슈

ex) #17

📝 작업 내용

이벤트 유저의 회원가입 및 인증 과정을 #14 에서 진행해주신 JWT 토큰과 결합하여 구현하였습니다.

  • (이름+전화번호)를 통해 DB에 유저가 존재하는지 판정하는 기능
  • 외부 API를 통해 유저의 전화번호에 인증번호를 발송하는 기능
  • 인증번호 맞히면 DB에 유저 데이터 저장하고 JWT 토큰 발급
  • 인증번호 틀릴 시 서버에서 Exception 발생
  • TDD 기반으로 단위테스트 먼저 작성

네이버 SMS API가 사업자 전용으로 전환됨에 따라, CoolSMS에서 제공하는 문자 발송 API를 부득이하게 활용하게 되었습니다. 이 역시 변동가능성이 있다고 판단하여 SmsService 인터페이스를 통해 확장성을 대비하고자 했습니다.

한편, 유저가 "인증 중"임을 알리는 추가적인 세션 등의 조치가 필요하여 추후 반영할 예정입니다.
또한, 발급된 JWT를 다시 서버가 받았을 때 아직 온전하게 추출되지 않는 문제가 있어 살펴볼 예정입니다.

참고 이미지 및 자료

image
CoolSMS Java SDK

💬 리뷰 요구사항

@win-luck win-luck added the feat 기능 구현 label Jul 31, 2024
@win-luck win-luck requested a review from blaxsior July 31, 2024 09:28
@win-luck win-luck self-assigned this Jul 31, 2024
@blaxsior
Copy link
Collaborator

유저 로그인부터 API 연동까지 진짜 많은 것들을 잘 해주셨네요! 프로젝트가 잘 진행되고 있는 것 같아 좋습니다! 현재 발급된 토큰에 문제가 조금 있는 것 같아 해결하는 과정에 있습니다. 이 부분은 내일까지 처리해볼게요.

@win-luck win-luck linked an issue Jul 31, 2024 that may be closed by this pull request
5 tasks
@win-luck win-luck merged commit 32125c5 into dev Jul 31, 2024
1 check passed
@win-luck win-luck deleted the feature/17-feat-login branch August 2, 2024 09:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat 기능 구현
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feat] 이벤트유저 로그인 기능 구현 (#17)
2 participants