Skip to content

fix: 인증 오류와 일정 상세 메타 개선 - #24

Merged
Heyaaz merged 4 commits into
mainfrom
refactor/backend-oauth-metadata-cleanup
May 29, 2026
Merged

fix: 인증 오류와 일정 상세 메타 개선#24
Heyaaz merged 4 commits into
mainfrom
refactor/backend-oauth-metadata-cleanup

Conversation

@Heyaaz

@Heyaaz Heyaaz commented May 29, 2026

Copy link
Copy Markdown
Owner

Summary

  • 인증 오류 응답 정합성 개선
  • 일정 상세 메타·참고군 문구 개선

Changes

  • 회원가입 중복·로그인 실패 HTTP 오류 응답 정정
  • 인증 계약 테스트 추가
  • 일정 상세 Google/TourAPI 메타 응답 매핑 보강
  • 참고군 수용도 점수 기준 통일
  • 내부 취향 반영 문구를 사용자 혜택 문구로 변환

Verification

  • Not run
  • Build
  • Test
  • Manual

Notes

  • 기존 생성 일정은 저장된 검증 JSON 재생성 필요

@Heyaaz Heyaaz left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Nogada PR Review

Verdict: Approved ✅

Blocking 이슈는 발견하지 못했습니다.

확인한 범위

  • OAuth 로그인 공통 응답 생성 리팩터링 및 중복 사용자 생성 흐름
  • 로컬 회원가입/로그인 실패 시 HTTP 409/401로 내려가도록 바뀐 인증 계약
  • 일정 상세 응답의 이미지 fallback 및 popularity 메타 노출
  • 참고군 검증 문구가 개인명/내부 reason을 덜 노출하도록 바뀐 로직
  • 추가/수정된 테스트 커버리지

검증

  • git diff --check origin/main...HEAD
  • ./gradlew clean test --no-daemon --no-build-cache
  • ✅ 테스트 XML 요약: 12 files / 70 tests / 0 failures / 0 errors / 0 skipped

참고: 테스트 종료 시 Testcontainers/Hikari shutdown warning이 있었지만 Gradle은 성공했고 XML 실패도 0건이라 비차단 로그로 판단했습니다.

@Heyaaz
Heyaaz merged commit 12be2e5 into main May 29, 2026
1 check passed
@Heyaaz
Heyaaz deleted the refactor/backend-oauth-metadata-cleanup branch May 29, 2026 07:39
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.

1 participant