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

401 인증 오류에서 500코드가 나가는 버그 수정 #163

Merged
merged 3 commits into from
Jul 31, 2024

Conversation

ksk0605
Copy link
Contributor

@ksk0605 ksk0605 commented Jul 31, 2024

PR의 목적이 무엇인가요?

401 인증 오류에서 500코드가 나가는 버그 수정

이슈 ID는 무엇인가요?

설명

  • AuthenticationCheckInterceptor 에서 AuthException을 발생하도록 수정
  • GlobarExceptionHandler 에서 AuthException 을 처리하도록 수정

질문 혹은 공유 사항 (Optional)

@ksk0605 ksk0605 added BE 백엔드 관련 이슈입니다. 🚑 버그 fix (develop에서 파생된 문제) labels Jul 31, 2024
@ksk0605 ksk0605 self-assigned this Jul 31, 2024
Copy link
Contributor

@Mingyum-Kim Mingyum-Kim 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

@ay-eonii ay-eonii left a comment

Choose a reason for hiding this comment

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

짱!

Comment on lines 30 to 33
@ExceptionHandler(AuthException.class)
public ResponseEntity<ErrorResponse> handleMoudaException(AuthException exception) {
return ResponseEntity.status(exception.getHttpStatus()).body(new ErrorResponse(exception.getMessage()));
}
Copy link
Contributor

Choose a reason for hiding this comment

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

MoudaException에서 잡힐 것 같아용~!

@ksk0605 ksk0605 merged commit 70e3f28 into develop-backend Jul 31, 2024
1 check passed
@ss0526100 ss0526100 deleted the feature/#161 branch August 1, 2024 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE 백엔드 관련 이슈입니다. 🚑 버그 fix (develop에서 파생된 문제)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants