Skip to content

[refactor/#428] Auth / Security 컨텍스트 소유 표면 정리#435

Merged
Dimo-2562 merged 10 commits into
developfrom
refactor/#428
Jun 14, 2026
Merged

[refactor/#428] Auth / Security 컨텍스트 소유 표면 정리#435
Dimo-2562 merged 10 commits into
developfrom
refactor/#428

Conversation

@Dimo-2562

@Dimo-2562 Dimo-2562 commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

❤️ 기능 설명

Auth / Security 컨텍스트의 실제 소유 표면을 auth 최상위 패키지로 정리했습니다.

  • domain/authauth/{presentation,application,domain,infrastructure} 구조로 승격
  • iOS 직접 Kakao 로그인 흐름을 KakaoLoginController / KakaoLoginService로 분리
  • 기존 global/security의 JWT, OAuth/OIDC, filter, handler, config, cookie util, auth cache를 auth/security 아래로 회수
  • Auth / Security 테스트 패키지를 현재 컨텍스트 구조에 맞춰 이동
  • JWT 예외 발생 시 기존 SecurityContext가 남지 않도록 SecurityContextHolder.clearContext() 적용
  • Auth / Security 문서와 테스트 인벤토리를 현재 구조 기준으로 최신화

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

  • API 계약 변경 없는 리팩터링 중심 PR이라 Swagger 스크린샷은 해당 없습니다.
  • 아래 CLI 검증 결과로 대체합니다.

연결된 issue

close #428



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

  • auth/security를 Auth / Security 컨텍스트 내부의 앱 전역 인증/인가 shared kernel로 두는 구조로 결정했습니다.

✅ 체크리스트

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

🧪 테스트 결과

git diff --check develop
# 통과

./gradlew test -PexcludeIntegration \
  --tests 'com.techfork.auth.*'
# BUILD SUCCESSFUL

./gradlew test \
  --tests 'com.techfork.domain.admin.controller.AdminControllerIntegrationTest.generateDeveloperToken_Fail_InvalidToken'
# BUILD SUCCESSFUL

@Dimo-2562 Dimo-2562 self-assigned this Jun 14, 2026
@Dimo-2562 Dimo-2562 added 🌟 REFACTOR 리팩터링 (기능 변화 X) 🔐 domain:auth 인증/보안 관련 (로그인, JWT, OAuth) labels Jun 14, 2026
@Dimo-2562 Dimo-2562 merged commit 7103c85 into develop Jun 14, 2026
1 check passed
@Dimo-2562 Dimo-2562 deleted the refactor/#428 branch June 14, 2026 12:50
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 / Security 컨텍스트 소유 표면 정리

1 participant