Feat/backoffice auth #40
Conversation
| return; | ||
| } | ||
| } else { | ||
| System.out.println("필터에서 username 파라미터를 가져올 수 없습니다."); |
|
|
||
| private final RedisConnectionFactory redisConnectionFactory; | ||
|
|
||
| @Bean |
There was a problem hiding this comment.
기본 StringRedisTemplate 써도 되지 않을까요?
|
|
||
| @Override | ||
| public UserDetails loadUserByUsername(String username) throws UsernameNotFoundException { | ||
| AdminUser admin = adminUserService.findOptAdminUser(username) |
There was a problem hiding this comment.
예외 반환을 data 모듈의 service에서 하지 않고 여기서 하신 이유가 있을까요~!
There was a problem hiding this comment.
앗! 깜박했습니다 감사합니다office-data 에서 처리하도록 하겠습니다
| private final AdminUserService adminUserService; | ||
|
|
||
| @Override | ||
| public UserDetails loadUserByUsername(String username) throws UsernameNotFoundException { |
There was a problem hiding this comment.
메서드 시그니처에는 UsernameNotFoundException을 던진다고 선언되어 있는데, 실제로는 NotFoundAdminUserException을 던지고 있어 인터페이스 계약과 일치하지 않는 것 같습니다! 커스텀 예외가 UsernameNotFoundException을 상속하도록 수정하거나 data service layer에서 에러 반환하는 방식으로 한뒤, 여기서 UsernameNotFoundException로 명시적으로 변환하는게 좋을 것 같아요
There was a problem hiding this comment.
아하 usernameNotFoundException을 상속하도록 수정하겠습니다 체크 감사합니다.
| @@ -0,0 +1,7 @@ | |||
| package com.clip.global.exception; | |||
|
|
|||
| public class NotExistAdminUserException extends RuntimeException { | |||
There was a problem hiding this comment.
요거 global/exception 하위에 두신 이유가 있을까요?
There was a problem hiding this comment.
해당 예외처리를global 의 LoginAttemptFilter에서 사용해서 해당 디렉토리에 넣엇습니다. 생각해보니 admin의 존재 유무를 체크하는 예외처리니 admin으로 리팩토링하겟습니다.
back office 로그인 구현