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

fix: jwt 인증 필터 #39

Merged
merged 9 commits into from
Jan 17, 2024
Merged

fix: jwt 인증 필터 #39

merged 9 commits into from
Jan 17, 2024

Conversation

seokho-1116
Copy link
Collaborator

작업 내용 (Content)

  • jwt 인증 필터가 특정 엔드포인트를 무시하도록 설정
  • Security Config 리팩토링 (@order 순서, JwtDsl에서 AuthenticationManager 가져오는 방식)
  • InvalidTokenException을 BusinessException으로 변경
  • 필터에서 발생하는 예외를 ErrorCode상 AuthenticationException에 관한 내용으로 반환하도록 변경

링크 (Links)

기타 사항 (Etc)

  • 기존의 필터는 특정 엔드포인트를 무시하지 못하는 버그가 있었다.

Merge 전 필요 작업 (Checklist before merge)

희망 리뷰 완료 일 (Expected due date)

- Error code에 status 추가
- Error response 필드 이름 변경
- BusinessException으로 상속 변경
- Security에서 예외를 가져가서 문제 발생해서 수정
- InvalidTokenException 상속 변경으로 인한 수정
- 인증 불필요한 엔드포인트 처리 가능하게 변경
- Failure handler 삭제, filter에서 처리하도록 변경
- BusinessException 생성자 추가
- 예외 핸들러 추가
@seokho-1116 seokho-1116 added the fix 버그 수정 label Jan 17, 2024
- 기존의 JwtAuthenticationFailureHandler의 삭제로 제거
- /status -> /me/status로 변경
- Security Config 수정
- Api 공통 응답 추가
Copy link
Collaborator

@GaBaljaintheroom GaBaljaintheroom left a comment

Choose a reason for hiding this comment

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

확인 하였습니다.

@GaBaljaintheroom GaBaljaintheroom merged commit 81a8f05 into develop_back Jan 17, 2024
2 checks passed
@seokho-1116 seokho-1116 deleted the ARCH-114-fix/jwt branch January 25, 2024 08:25
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.

2 participants