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

시큐리티 업데이트 관련 리팩토링 #187

Open
2 tasks
vectorch9 opened this issue Aug 1, 2022 · 1 comment
Open
2 tasks

시큐리티 업데이트 관련 리팩토링 #187

vectorch9 opened this issue Aug 1, 2022 · 1 comment

Comments

@vectorch9
Copy link
Member

vectorch9 commented Aug 1, 2022

Feature Description

  • 스프링 시큐리티 업데이트로 바뀐 사항 적용하기

최근에 Security 업데이트가 있었는데 해당 부분 리팩토링 해보시는 것도 괜찮을 것 같아요!
테스트 코드 관련해서는 BBDMockito/JUnit5랑 7 Popular Unit Test Naming Convetions
, Give-When-Then Pattern 두 가지 포스팅도 참고해 보시면 좋을 것 같습니다!

Originally posted by @seonpilKim in #183 (comment)

Todo list

  • 관련 업데이트 내용 조사
  • 인증 관련 리팩토링
@vectorch9 vectorch9 self-assigned this Aug 1, 2022
@vectorch9 vectorch9 changed the title 최근에 [Security 업데이트](https://spring.io/blog/2022/02/21/spring-security-without-the-websecurityconfigureradapter)가 있었는데 해당 부분 리팩토링 해보시는 것도 괜찮을 것 같아요! 시큐리티 업데이트 관련 리팩토링 Aug 1, 2022
@vectorch9
Copy link
Member Author

Provider 설정 관련

  • 현재 프로젝트 filter 구현 과정에서 여러 개의 provider를 설정해줘야합니다. 기존엔 super.authenticationManager()를 통해 쉽게 등록할 수 있었지만, Adapter를 상속하지 않기 때문에 문제가 발생했습니다.
  • 해당 포스트의 댓글 최상단에도 같은 문제를 5달전에 언급하였으나 아직까지 해결방법을 찾지 못했다고 합니다.
  • 저도 관련에서 자료 조사를 진행해보았으나, 아무래도 올해 변경된 업데이트다 보니 관련 자료가 많지 않습니다.
  • 추가로, filter에 AuthenticationManager를 주입할 때도 super.authenticationManager()를 사용하였는데, 이 역시 사용이 불가능해졌습니다. (Autowired, RequiredArgsConstructor를 통해선 정상적으로 주입이 되지않습니다.)
  • 해당 이슈 해결 전 까진 시큐리티 Adapter Deprecaed 반영은 보류해야 할 것 같습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant