Skip to content

Conversation

@joowojr
Copy link
Contributor

@joowojr joowojr commented Feb 3, 2025

📄 요약(Summary)

인증 경로의 인증 오류에 대한 필터 체인 수정 및 예외 처리 로깅 개선

✍🏼 상세(More)

PR Desciption

변경 사항 설명

⛔️ 인증 및 인가가 필요한 엔드포인트에서 토큰 없이 요청 시에, 401이 아닌 500에러 발생
   기존 시큐리티 필터에 로깅 및 감사를 위한 response body 캐싱 필터가 추가되어
   토큰이 없는 요청에 대해 캐싱 필터로 넘어가 api를 호출하게 되어 이 api를 처리하던 과정 중에 500에러가 발생.
  • 인증이 필요한 요청에 대해서, 액세스 토큰이 없을 경우 401 예외를 던지도록 수정

  • 예외 처리 로깅 개선

  • adapter/inbound/security 디렉토리 구조 정리

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요
ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

PR Log

PR 작업하면서 고민했던 내용, 해결한 내용, 고민 중인 내용 등

새롭게 배운 것

고민 중인 사항

  • 현재 dto validation에 대해서 실패하는 경우에 감사 로깅이 찍히지 않는 것 같은데 이에 대해서도 로깅을 남겨야 할지에 대해 고민이 됩니다.

첨부 자료

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

✅ 체크리스트(Checklist)

  • PR 양식에 맞게 작성했습니다
  • 모든 테스트가 통과했습니다
  • 프로그램이 정상적으로 작동합니다
  • 적절한 PR 라벨을 설정했습니다
  • 불필요한 코드를 제거했습니다

🚪 이슈 번호(Issue numbers)

Closes #269

@joowojr joowojr self-assigned this Feb 3, 2025
@joowojr joowojr added the 🐛 bug 버그를 수정합니다 label Feb 3, 2025
@joowojr joowojr merged commit 3c90b31 into develop Feb 3, 2025
1 check passed
@joowojr joowojr deleted the CLAP-251 branch February 3, 2025 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 bug 버그를 수정합니다

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CLAP-251 인증이 요구되는 요청에 대한 필터 체인 로직 오류 수정

2 participants