benelog / contents-proxy

4 stars 1 forks source link

Spring interceptor 를 이용한 관리자 로그인 기능 구현 PR #33

Closed jiwonbabe closed 5 years ago

jiwonbabe commented 5 years ago

여기도 마찬가지로 매번 새로운 BCryptPasswordEncoder 을 생성하고 있습니다. 이 경우에 멤버변수로 빼려고 하면 Account 클래스가 Entity이므로 적절하지 않게 되는데요. 암호화 기능에 관계된 것이 Account 에 필요한 기능인지 고민해볼 필요가 있을 것 같습니다. Account에서만 쓰이는 기능이 아니라면 암(복)호화 기능을 위한 별도의 유틸리티 혹은 컴포넌트를 만드는 것이 어떨까요?

피드백 감사합니다. 그런데 제가 오늘 로직을 많이 바꿔서 삭제한 코드가 많습니다. 바뀐 코드로 혹시 봐주실 수 있나요?