Skip to content

#603 [Fix] JwtFilter 계정 상태 검사에서 정상 계정이 401로 막히는 문제 수정 - #604

Merged
si-zero merged 1 commit into
devfrom
fix/#603
Nov 2, 2026
Merged

si-zero merged 1 commit into
devfrom
fix/#603

Conversation

@si-zero

@si-zero si-zero commented Nov 2, 2026

Copy link
Copy Markdown
Member

#️⃣ 연관된 이슈

📝 작업 내용

🐛 Fix

내용 파일
계정 조회와 상태 판정을 분리해 ACTIVE·PENDING 계정은 통과시키도록 수정 JwtFilter.java
계정 상태별(통과·403·401) 필터 테스트 추가 JwtFilterTest.java

📌 공유 사항

  1. ✨ Feat: 회원 탈퇴 시 클래스·좌석·콘텐츠 처리 #584 의 accountError 가 Optional.map 안에서 정상 계정일 때 null 을 돌려줘서 빈 값이 되고, 뒤의 orElse(AUTH_UNAUTHORIZED) 가 적용되고 있었습니다. 그래서 정상 사용자의 인증 요청이 모두 401(AUTH_401)이 됐습니다.
  2. 클래스·토론·주제 통합 테스트가 이 문제로 전부 실패하고 있었고, dev CI 실패 원인 중 하나입니다.
  3. 이 PR 이후에도 dev 에 아래 테스트 실패 34건이 남아 있습니다. dev 원본에서도 똑같이 실패하며 이번 수정과는 관계없습니다.
    • ClassroomServiceImplTest, ClassMemberServiceImplTest: 서비스가 사용자를 먼저 조회하게 바뀌었는데 테스트에 사용자 모킹이 없어 USER_NOT_FOUND
    • AdminFundingApiIntegrationTest, TumblbugFundingApiIntegrationTest: 503 응답과 발급 건수 불일치
    • OAuthServiceTest, PerspectiveServiceTest: CustomException

🧪 테스트

  • ./gradlew test --tests '*JwtFilterTest' - 성공
  • 전체 테스트: 625건 중 34건 실패 (모두 위 3번의 기존 실패)

✅ 체크리스트

  • 컨벤션에 맞는 Type을 선택했나요?
  • Merge 하려는 브랜치가 올바르게 설정되어 있나요?
  • 로컬에서 실행했을 때 에러가 발생하지 않나요?

@si-zero si-zero self-assigned this Nov 2, 2026
@si-zero si-zero added the 🐛 Fix (버그 수정) 버그를 수정할 때 사용합니다. label Nov 2, 2026
@si-zero
si-zero merged commit 9770918 into dev Nov 2, 2026
1 check failed
@si-zero
si-zero deleted the fix/#603 branch November 2, 2026 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 Fix (버그 수정) 버그를 수정할 때 사용합니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant