Skip to content

[refactor/#430] Auth cache 이벤트 seam 안정화#439

Merged
Dimo-2562 merged 3 commits into
developfrom
refactor/#430
Jun 15, 2026
Merged

[refactor/#430] Auth cache 이벤트 seam 안정화#439
Dimo-2562 merged 3 commits into
developfrom
refactor/#430

Conversation

@Dimo-2562

Copy link
Copy Markdown
Contributor

❤️ 기능 설명

Auth cache 이벤트 seam을 보안 정책 관점에서 명확히 고정했습니다.

  • OnboardingCompletedEvent 후 auth cache evict가 AFTER_COMMIT으로만 실행되는 정책을 테스트로 고정
  • UserWithdrawnEvent 후 auth cache evict가 BEFORE_COMMIT + AFTER_COMMIT으로 실행되는 정책을 테스트로 고정
  • 탈퇴 BEFORE_COMMIT auth cache evict 실패 시 탈퇴 트랜잭션이 rollback되는 경로를 통합 테스트로 보강
  • UserAuthCacheEventListener 핸들러명을 이벤트/트랜잭션 phase가 드러나도록 정리
  • UserAuthCacheServiceTestget / put / evict Nested 구조로 정리하고 기존 wire format 유지 검증 보강
  • HttpCookieOAuth2AuthorizationRequestRepositoryTestsaveAuthorizationRequest / loadAuthorizationRequest / removeAuthorizationRequest Nested 구조로 정리

Swagger 테스트 성공 결과 스크린샷 첨부

  • API path, JSON field, status code 변경 없는 Auth/Security 리팩터링 및 테스트 보강이라 Swagger 스크린샷은 해당 없습니다.
  • 아래 CLI 검증 결과와 PR CI로 대체합니다.

연결된 issue

close #430



🩷 Approve 하기 전 확인해주세요!

  • 탈퇴 BEFORE_COMMIT auth cache evict 실패는 의도적으로 탈퇴 트랜잭션 rollback을 유발하는 fail-closed 보안 정책입니다.
  • 이 정책은 탈퇴 사용자 인증 차단을 우선하지만, Redis delete 실패가 탈퇴 실패로 노출될 수 있는 운영 tradeoff가 있습니다.

✅ 체크리스트

  • PR 제목 규칙 잘 지켰는가?
  • 추가/수정사항을 설명하였는가?
  • 테스트 결과 사진을 넣었는가? (API 계약 변경 없는 리팩터링이라 CLI 결과로 대체)
  • 이슈넘버를 적었는가?

🧪 테스트 결과

git diff --check
# 통과

./gradlew test -PexcludeIntegration
# BUILD SUCCESSFUL

./gradlew test --tests 'com.techfork.useraccount.integration.UserAccountAfterCommitEventIntegrationTest'
# BUILD SUCCESSFUL

🔎 리뷰 결과

$code-review
- code-reviewer recommendation: APPROVE
- architect status: WATCH
- final recommendation: COMMENT
  • 추가 수정 필수 사항은 발견되지 않았습니다.
  • WATCH 항목은 위 Approve 전 확인사항의 Redis fail-closed 운영 tradeoff입니다.

@Dimo-2562 Dimo-2562 self-assigned this Jun 15, 2026
@Dimo-2562 Dimo-2562 added 🌟 REFACTOR 리팩터링 (기능 변화 X) 🔐 domain:auth 인증/보안 관련 (로그인, JWT, OAuth) labels Jun 15, 2026
@Dimo-2562 Dimo-2562 merged commit fae8d77 into develop Jun 15, 2026
1 check passed
@Dimo-2562 Dimo-2562 deleted the refactor/#430 branch June 15, 2026 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🔐 domain:auth 인증/보안 관련 (로그인, JWT, OAuth) 🌟 REFACTOR 리팩터링 (기능 변화 X)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[REFACTOR] Auth cache 이벤트 seam 안정화

1 participant