Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Modify UsernameNotFoundException Error handling #134

Merged
merged 5 commits into from
Oct 22, 2023

Conversation

inh2613
Copy link
Member

@inh2613 inh2613 commented Oct 22, 2023

PR Type

  • 기능 추가
  • 기능 삭제
  • 버그 수정
  • 테스트 코드 작성
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트
  • 문서 작성

Motivation

탈퇴한 회원으로 내정보 조회 시, 500에러 발생
image

Problem Solving

최상단 Filter인 JwtAuthenticationFilter에서 UsernameNotFoundException 핸들링 하도록 하여 401 에러를 응답으로 보내게 함

To Reviewer

관련 자료 참고

@inh2613 inh2613 added 🐛 bug Something isn't working 👮‍♂️ Auth 회원 및 인증에 관련된 라벨 labels Oct 22, 2023
@inh2613 inh2613 self-assigned this Oct 22, 2023
Copy link
Member

@jinlee1703 jinlee1703 left a comment

Choose a reason for hiding this comment

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

확인했습니다.

Copy link
Contributor

@seheon99 seheon99 left a comment

Choose a reason for hiding this comment

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

확인했습니다!

@inh2613 inh2613 merged commit 69128bf into develop Oct 22, 2023
2 checks passed
@inh2613 inh2613 deleted the fix/GH-354-user-info-read-error branch October 22, 2023 08:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
👮‍♂️ Auth 회원 및 인증에 관련된 라벨 🐛 bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants