Skip to content

Conversation

@minhyuk2
Copy link
Member

@minhyuk2 minhyuk2 commented Jun 10, 2025

#️⃣ Issue Number


📝 요약 (Summary)

  • SecurityCORS 기본 설정을 완료했습니다.
    • CORS는 추후 프론트 주소가 확정되면 수정 예정입니다.
  • OAuth2를 활용한 구글 로그인 기능을 구현했습니다.

🔐 로그인 동작 방식

  1. 프론트엔드에서 구글 소셜 로그인 URL로 리디렉션
  2. 인가 코드(code)를 받아 백엔드에 전달
  3. 백엔드에서 다음과 같이 처리
    • 가입된 회원인 경우: 로그인 처리
    • 가입되지 않은 회원인 경우:
      • 회원가입이 필요하다는 메시지와 함께 이메일 주소를 반환
      • 이때 PENDING 상태로 유저 정보를 DB에 저장
      • 이후 프론트에서 이름, wallet_uuid, 약관 동의 등의 정보를 받아 회원가입 API 호출 시, 유저 상태를 SUCCESS로 업데이트

💬 공유사항 (To Reviewer)

  • CORS 설정과 인증 흐름 관련하여 피드백 주시면 감사하겠습니다!

✅ PR Checklist

  • 커밋 메시지를 컨벤션에 맞게 작성했습니다.
  • 변경된 기능에 대한 테스트를 완료했습니다.

@minhyuk2 minhyuk2 requested a review from hyuneu-n June 10, 2025 08:31
@minhyuk2 minhyuk2 self-assigned this Jun 10, 2025
Copy link
Contributor

@rtaeho rtaeho left a comment

Choose a reason for hiding this comment

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

Webflux 환경에서 구글 로그인 붙이는 거 쉽지 않았을 텐데 고생 많았네
전체적으로 구조 잘 나눴고, CORS나 JWT 인증 처리도 깔끔하게 잘 한 것 같아!

@hyuneu-n hyuneu-n merged commit 17b6a5c into dev Jul 23, 2025
rtaeho added a commit that referenced this pull request Aug 29, 2025
JWT 검증 부분 임시 주석 처리
rtaeho added a commit that referenced this pull request Sep 4, 2025
* feat: #7 달 별로 경제지표 반환하는 api 작성

* feat: #5 뉴스 상위 10개 반환 API 작성

JWT 검증 부분 임시 주석 처리

* feat: #7 뉴스 상위 10개 반환 API 작성

JWT 검증 부분 임시 주석 처리
rtaeho added a commit that referenced this pull request Sep 4, 2025
* feat: #7 달 별로 경제지표 반환하는 api 작성

* feat: #5 뉴스 상위 10개 반환 API 작성

JWT 검증 부분 임시 주석 처리

* feat: #7 뉴스 상위 10개 반환 API 작성

JWT 검증 부분 임시 주석 처리

* �fix: #10 오타 수정
rtaeho added a commit that referenced this pull request Sep 4, 2025
* feat: #7 달 별로 경제지표 반환하는 api 작성

* feat: #5 뉴스 상위 10개 반환 API 작성

JWT 검증 부분 임시 주석 처리

* feat: #7 뉴스 상위 10개 반환 API 작성

JWT 검증 부분 임시 주석 처리

* �fix: #10 오타 수정

* fix: #10 JWT 검증 부분 주석 해제
rtaeho added a commit that referenced this pull request Sep 12, 2025
* feat: #7 달 별로 경제지표 반환하는 api 작성

* feat: #5 뉴스 상위 10개 반환 API 작성

JWT 검증 부분 임시 주석 처리

* feat: #7 뉴스 상위 10개 반환 API 작성

JWT 검증 부분 임시 주석 처리

* �fix: #10 오타 수정

* fix: #10 JWT 검증 부분 주석 해제

* feat: swagger 작성
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants