Skip to content

fix: 초대 참여 인증 계약 정리 - #17

Merged
Heyaaz merged 1 commit into
mainfrom
feature/invite-auth-mypage
May 17, 2026
Merged

fix: 초대 참여 인증 계약 정리#17
Heyaaz merged 1 commit into
mainfrom
feature/invite-auth-mypage

Conversation

@Heyaaz

@Heyaaz Heyaaz commented May 17, 2026

Copy link
Copy Markdown
Owner

Summary

  • 초대 참여 게스트 차단
  • 로그인 계정 기반 내 여행 계획 조회 계약 보강
  • OAuth 초대 redirectPath 보존 검증

Changes

  • 게스트 세션 join 요청 403 반환
  • /rooms/my 참여 멤버 방 목록 포함 검증
  • 초대 join 호환 엔드포인트 게스트 차단 검증
  • AuthContractTests 중복 케이스 정리

Verification

  • Not run
  • Build
  • Test: ./gradlew test --tests 'com.tripsync.AuthContractTests'
  • Manual

Notes

@Heyaaz

Heyaaz commented May 17, 2026

Copy link
Copy Markdown
Owner Author

Nogada PR Review

Verdict

  • 승인 가능: blocking 이슈 없음

Blocking issues

  • 없음

Non-blocking notes

  • 테스트 종료 시 Testcontainers/Hikari PostgreSQL 연결 종료 경고가 로그에 남았지만, Gradle 결과와 XML 기준 실패는 없음.

Checked scope

  • 초대 참여 시 guest 계정 차단 로직 확인
  • /rooms/my가 방장 전용 문구가 아니라 로그인 계정의 참여 방 목록 계약에 맞게 동작하는지 확인
  • 초대 참여 후 멤버도 홈 목록에서 방을 조회하는 계약 테스트 확인
  • OAuth join redirect path 보존 테스트 확인
  • git diff --check origin/main...HEAD 통과
  • /tmp/tripsync_server-pr-17 worktree에서 ./gradlew clean test --no-daemon --no-build-cache 실행: 47 tests, 0 failures, 0 errors

@Heyaaz
Heyaaz merged commit 69eb877 into main May 17, 2026
1 check passed
@Heyaaz
Heyaaz deleted the feature/invite-auth-mypage branch May 18, 2026 04:25
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