PR :: SecurityFilterChain에서 발생한 Authentication 관련 예외를 ErrorLogResponseFilter에서 처리하도록 수정

[Daybreak312]

어떤 종류의 PR 인가요?

/kind 기능
/kind 리펙토링

이 PR이 무슨 일을 하나요? / 필요한 이유가 뭔가요?

SecurityFilterChain 실행 중에 인증이 실패하면, AccessDeniedHandler 혹은 AuthenticationEntryPoint로 Handle되는데, 이렇게 Handle된 예외를 해당 클래스에서 처리하지 않고 AuthenticationException으로만 전환하고 ErrorLogResponseFilter에서 처리하도록 했습니다.

또한, 이 PR에는 #186 의 변경 사항이 포함되어 있습니다.

리뷰어를 위한 참고사항:

1. AccessDeniedHandler에 있던, 이미 에러 페이지가 있을 경우에 대한 로직을 제거한 이유
Exception이 발생하고, 이를 Spring에서 처리해 errorPage를 로딩할 경우 사용자가 Controller까지 가기 위한 여정을 다시 경험하게 됩니다.
이러한 부분을 불필요하고, 비효율적인 로직이라 판단했기에 마음가짐에서 Spring의 ErrorPage를 제공하는 경우를 모두 없애려 합니다.
이러한 제 견해를 실행에 옮기기 위한, 기나길 리펙토링 여행의 첫 걸음입니다.

사실 이 변경점과 앞서 언급한 문제점과는 별 연관이 없지만, 그래도 에러 페이지의 존재 가능성을 제거하고자 해당 로직 또한 제거했습니다.

---

• To see the specific tasks where the Asana app for GitHub is being used, see below:
  • https://app.asana.com/0/0/1206937090191548