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.

Sign up for GitHub

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

Jump to bottom

[BSVR-71] 로그인/회원가입, JWT 구현 #34

Merged
merged 48 commits into from
Jul 18, 2024
Merged

[BSVR-71] 로그인/회원가입, JWT 구현 #34

merged 48 commits into from
Jul 18, 2024

Conversation

wjdwnsdnjs13
Copy link
Member

📌 개요 (필수)

  • 카카오를 사용한 로그인/회원가입을 진행할 수 있어요
  • JWT와 필터를 구현했어요

🔨 작업 사항 (필수)

  • 회원가입 API
  • 로그인 API
  • 닉네임 중복 검사 API
  • JWT
  • JWT Filter

⚡️ 관심 리뷰 (선택)

  • conflict가 발생했는데, 제 윈도우에선 되는 데, 다른 환경에서도 돌아가는 지 모르겠어요. 확인이 필요해요
  • 슬랙에 오늘 결산에 보낸 환경 변수들을 .env 파일안에 넣어주세요!
  • DDL 수정이 필요해요.
    • email, name, phone_number 을 nullable로 변경해야해요.
    • level을 nullable, default value 1 로 변경해야해요.
  • 애플리케이션 실행 후 API들이 정상 동작하는지 확인이 필요해요.

💻 실행 화면 (필수)

  • 회원 가입
    image

  • 로그인
    image

  • 닉네임 중복 확인
    image

@wjdwnsdnjs13
build : security 의존성 추가
37bce49
@wjdwnsdnjs13
feat : security config 추가
b3db4db
@wjdwnsdnjs13
docs : jwt 시크릿 키 추가
2cd496c
@wjdwnsdnjs13
build : jwt 사용을 위한 라이브러리 추가
279afa5
@wjdwnsdnjs13
feat : jwtFilter 등록
8cb0cd5
@wjdwnsdnjs13
feat : jwt 구현
76aedce
@wjdwnsdnjs13
feat : jwtFilter 구현
7ba6913
@wjdwnsdnjs13
docs : jwt 구현
ed3a206
@wjdwnsdnjs13
docs : 카카오 Http 요청을 위한 webflux 추가
20888cd
@wjdwnsdnjs13
feat : member 도메인 구현
52685e2
@wjdwnsdnjs13
feat : memberEntity 구현
4e3ff59
@wjdwnsdnjs13
feat : 카카오 엑세스 토큰, 유저 정보 가져오기 구현
37e56d9
@wjdwnsdnjs13
feat : 멤버 Role 관리를 위한 enum 구현
b227d2c
@wjdwnsdnjs13
remove : 초기 세팅 테스트용으로 작성된 코드 제거
a6696a0
@wjdwnsdnjs13
feat : 닉네임 중복 확인 구현
1cec21b
@wjdwnsdnjs13
feat : 닉네임 중복 확인 구현
94e99f9
@wjdwnsdnjs13
feat : 닉네임 중복 에러 구현
3b6cf1f
@wjdwnsdnjs13
refactor : 카카오 계정 정보 가져오는 로직 변경
781679e
@wjdwnsdnjs13
feat : 회원가입, 로그인 구현
06093ba
@wjdwnsdnjs13
feat : 회원가입, 로그인 구현
42cb1ba
@wjdwnsdnjs13
feat : id 토큰을 이용한 로그인 구현
69d8c81
@wjdwnsdnjs13
refactor : email과 이름, 전화번호, 레벨 권한 문제로 인해 nullable 처리
8027695
@wjdwnsdnjs13
refactor : url과 clientId 키 값 분리
6191127
@wjdwnsdnjs13
feat : Authentication를 상속받은 jwt 토큰 구현
2108222
@wjdwnsdnjs13
refactor : 권한 문제로 인해 idToken값 비교로 변경
eecd86f
@wjdwnsdnjs13
docs : oauth를 위한 환경 변수 추가
97e5d22
@wjdwnsdnjs13
fix : SecretKey 타입 차이로 인한 공백 오류 해결
0ad7e30
@wjdwnsdnjs13
refactor : 로그인, 회원가입 JWT 사용해서 구현
c555073
@wjdwnsdnjs13
Merge remote-tracking branch 'origin/main' into feat/BSVR-71
20bdfe6 
# Conflicts:
#	domain/src/main/java/org/depromeet/spot/domain/member/Member.java
#	infrastructure/jpa/src/main/java/org/depromeet/spot/jpa/member/entity/MemberEntity.java
#	usecase/src/main/java/org/depromeet/spot/usecase/service/MemberService.java
@wjdwnsdnjs13
feat : 빌더 패턴 추가
3232ffa
@github-actions GitHub Actions
Copy link

github-actions bot commented Jul 17, 2024 •
edited
Loading

Test Results

44 tests  +5   44 ✅ +5   0s ⏱️ ±0s
16 suites +2    0 💤 ±0 
16 files   +2    0 ❌ ±0 

Results for commit 66d769d. ± Comparison against base commit 952a226.

♻️ This comment has been updated with latest results.

@pminsung12
Copy link
Collaborator

pminsung12 commented Jul 17, 2024 •
edited
Loading

엄청나다 1400줄, 수고많았어😢 리뷰 시작한다잇!

@EunjiShin
Copy link
Collaborator

EunjiShin commented Jul 17, 2024 •
edited
Loading

DDL 수정이 필요해요.
email, name, phone_number 을 nullable로 변경해야해요.
level을 nullable, default value 1 로 변경해야해요.

@wjdwnsdnjs13 요건 저번에 얘기한대로, 카카오톡 비즈앱 생성되면 너가 entity 설정 바꿀 계획이라는 뜻이 맞을까?!

@EunjiShin
Copy link
Collaborator

EunjiShin commented Jul 17, 2024 •
edited
Loading

스크린샷 2024-07-17 오후 5 59 51

@wjdwnsdnjs13 ?! 여기 브랜치에서 스웨거 안돌아가는 이슈 발견..! 👀

@EunjiShin
Copy link
Collaborator

스크린샷 2024-07-17 오후 6 02 47

@wjdwnsdnjs13 알려준 가이드 라인 대로, idCode 발급 받은 다음 회원 가입 API 호출했는데 500 에러 발생..!

스크린샷 2024-07-17 오후 6 04 26 로그 보니까 NPE 이슈인데, 확인 부탁해!

@EunjiShin EunjiShin added the ❌ do not merge 아직 merge하면 안되는 상태 label Jul 17, 2024
@wjdwnsdnjs13
Copy link
Member Author

엄청나다 1400줄, 수고많았어😢 리뷰 시작한다잇!

ㅋㅋㅋ 하다 보니까 줄이 많아지더라...

@wjdwnsdnjs13
Copy link
Member Author

DDL 수정이 필요해요.
email, name, phone_number 을 nullable로 변경해야해요.
level을 nullable, default value 1 로 변경해야해요.

@wjdwnsdnjs13 요건 저번에 얘기한대로, 카카오톡 비즈앱 생성되면 너가 entity 설정 바꿀 계획이라는 뜻이 맞을까?!

응응 맞아!
지금 엔티티랑 kakao에서 userInfo 받아오는 Dto도 일단 지금은 null 값으로 받아오지만, 비즈앱 되면 금방 적용가능하게 해놨어!

@wjdwnsdnjs13
Copy link
Member Author

스크린샷 2024-07-17 오후 6 02 47 @wjdwnsdnjs13 알려준 가이드 라인 대로, idCode 발급 받은 다음 회원 가입 API 호출했는데 500 에러 발생..!

스크린샷 2024-07-17 오후 6 04 26 로그 보니까 NPE 이슈인데, 확인 부탁해!

헐... 오키오키 확인해볼게

@wjdwnsdnjs13 wjdwnsdnjs13 changed the base branch from dev to main July 17, 2024 10:01
@github-actions github-actions bot added size/L and removed size/XL labels Jul 17, 2024
@EunjiShin EunjiShin merged commit 50dcd96 into main Jul 18, 2024
6 checks passed
@EunjiShin EunjiShin deleted the feat/BSVR-71 branch July 18, 2024 12:56
@wjdwnsdnjs13 wjdwnsdnjs13 mentioned this pull request Aug 9, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@EunjiShin EunjiShin Awaiting requested review from EunjiShin EunjiShin is a code owner

@pminsung12 pminsung12 Awaiting requested review from pminsung12 pminsung12 is a code owner

Assignees

@wjdwnsdnjs13 wjdwnsdnjs13

Labels
❌ do not merge 아직 merge하면 안되는 상태 ✨ Feature 기능 개발 size/L
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@wjdwnsdnjs13 @pminsung12 @EunjiShin