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.

Sign up for GitHub

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

Jump to bottom

시큐리티 관련 코드 리팩토링 #19

Open
2 of 3 tasks
seonpilKim opened this issue Jul 1, 2022 · 0 comments · Fixed by #20
Open
2 of 3 tasks

시큐리티 관련 코드 리팩토링 #19

seonpilKim opened this issue Jul 1, 2022 · 0 comments · Fixed by #20
Assignees
@seonpilKim
Labels
☄ Refactor 코드 리팩토링

Comments

@seonpilKim
Copy link
Member

seonpilKim commented Jul 1, 2022 •
edited
Loading

Feature Description

  • 시큐리티 관련 코드 중 불필요한 코드를 제거하고, 빠뜨린 코드를 추가하는 전반적인 리팩토링 작업
  • 시큐리티 5.7.0 버전으로 마이그레이션

    Reference

Todo list

  • AuthenticationFailureHandler 변경: AuthenticationEntryPointFailureHandler
  • JwtAuthenticationFilter 리팩토링
  • 시큐리티 5.7.0 버전으로 마이그레이션
@seonpilKim seonpilKim added the ☄ Refactor 코드 리팩토링 label Jul 1, 2022
@seonpilKim seonpilKim self-assigned this Jul 1, 2022
seonpilKim added a commit that referenced this issue Jul 1, 2022
@seonpilKim
Feat: 인증 필터의 AuthenticationFailureHandler 변경
894f8a6 
AbstractAuthenticationProcessingFilter 클래스의 AuthenticationFailureHandler는 디폴트로 SimpleUrlAuthenticationFailureHandler을 사용한다.

AuthenticationEntryPointFailureHandler을 사용하도록 코드를 추가하였다.

Resolve: #19
seonpilKim added a commit that referenced this issue Jul 1, 2022
@seonpilKim
Refactor: JwtAuthenticationFilter 리팩토링
400c818 
다른 부모 생성자를 호출함으로써, 불필요한 재정의 코드 삭제

인증 성공 시, 디폴트 인증 성공 핸들러인 SavedRequestAwareAuthenticationSuccessHandler을 호출하지 않고, 단순히 인증 객체를 SecurityContextHolder에 저장한 후 다음 필터를 호출하는 로직으로 변경

인증 실패 시, AuthenticationEntryPointFailureHandler를 사용하기 위해 부모의 FailureHandler를 가져와서 onAuthenticationFailure 메소드를 호출하는 로직으로 변경

Resolve: #19
@seonpilKim seonpilKim mentioned this issue Jul 1, 2022
Merged
3 tasks
@seonpilKim seonpilKim linked a pull request Jul 1, 2022 that will close this issue
Security 관련 코드 리팩토링 & 카카오 로그인 방식 제거 #20
Merged
3 tasks
@seonpilKim seonpilKim mentioned this issue Jul 17, 2022
Merged
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@seonpilKim seonpilKim

Labels
☄ Refactor 코드 리팩토링
Projects
None yet
Milestone
No milestone
1 participant
@seonpilKim