Skip to content

Conversation

@simount3
Copy link
Collaborator

@simount3 simount3 commented Oct 13, 2025

제목(필수): [TYPE]: 제목 예) [FEAT]: 회원가입 기능 추가

제목 규칙 자세히 보기
  • 형식: [TYPE]: 제목
  • 제한: 50자 이내, 첫 글자 대문자, 명령문
  • TYPE: FEAT FIX REFACTOR COMMENT STYLE TEST CHORE INIT

무엇을 / 왜

  • 무엇(What):
    • GitHub OAuth2 로그인 시, 기본 응답에 이메일이 없을 경우 GitHub API를 추가 호출하여 이메일을 자동으로 가져오도록 개선
    • StableDiffusionImageClient에서 web client 빈 모호성 오류 -> WebClient 설정을 독립 구성
    • OAuth2 로그인 success 및 failure handler 프론트와 상의한 메인페이지로 redirect하도록 수정
    • production samesite도 Lax로 설정
  • 왜(Why):
    • 깃허브 퍼블릭 이메일이 설정되어 있지 않으면 email에 null 값이 들어옴 -> email 필드가 nullable false이기 때문에 오류
    • WebClient 빈 충돌 문제 해결

어떻게(요약) — 3줄 이내

  • GithubEmailFetcher 추가 → /user/emails API로 verified/primary 이메일 조회

  • StableDiffusionImageClient에 @qualifier("stabilityWebClient") 적용

  • OAuth2 실패 시 코드 매핑(EMAIL_MISSING, ACCESS_DENIED 등) 추가

  • 성공 핸들러 redirect URL 프론트 메인페이지

영향 범위

  • API 변경
  • DB 마이그레이션
  • Breaking Change
  • 보안/권한 영향
  • 문서/가이드 업데이트 필요

체크리스트

  • 타입 라벨 부착 (FEAT/FIX/REFACTOR/COMMENT/STYLE/TEST/CHORE/INIT)
  • 로컬/CI 테스트 통과
  • 영향도 점검 완료
  • 주석/문서 반영(필요 시)

ToDo (선택)

  • 할 일 1
  • 할 일 2

스크린샷/증빙(선택)

이슈 연결 (자동)

Closes #107

Copy link
Collaborator

@johnbosco0414 johnbosco0414 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

Copy link
Collaborator

@lcs9317 lcs9317 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다

@simount3 simount3 merged commit e6dad65 into main Oct 13, 2025
3 checks passed
@simount3 simount3 deleted the oauth branch October 13, 2025 03:27
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.

[FEAT]: GitHub 이메일 자동 조회

4 participants