#329 로그인 리졸버 유저 검증 로직 추가

[sojungpp]

📍 이슈 번호

• close [feature] IsLogin 리졸버 유저 검증 로직 추가 #329

🛠️ 작업 내용

1. ExceptionAdvice.java @RestControllerAdvice 추가

어노테이션 누락되어 있어서 추가했습니다. f906c54

2. 실행 오류 수정

develop 브랜치 실행했더니 오류가 뜨길래 임의로 해결했습니다. 참고링크 fd32abe

3. 유저 검증 로직 추가

• 로그아웃 / 회원탈퇴 확인 (Redis에 토큰 있는지 유무 확인)
  

  IceButler_Server/src/main/java/com/example/icebutler_server/global/util/TokenUtils.java

  Lines 262 to 264 in 078fa5b

  	public boolean isTokenExists(String key) {
  	return redisTemplateService.hasKey(key);
  	}

• 존재유저 / 중지유저 확인
  

  IceButler_Server/src/main/java/com/example/icebutler_server/user/service/UserServiceImpl.java

  Lines 181 to 184 in 078fa5b

  	public void validateUser(Long userId) {
  	if(!userRepository.existsByIdAndIsEnable(userId, true)) throw new BaseException(NOT_FOUND_USER);
  	if(!userRepository.existsByIdAndIsDenied(userId, false)) throw new BaseException(BLOCKED_USER);
  	}

4. LoginResolver return값 UserId(Long type)로 변경

기존 LoginStatus 클래스를 통해 userId를 가져오는 방식에서 userId로만 반환하는 로직으로 변경했습니다.

🎸 기타 사항

1. LoginResolver에서 Service 상속과 Repository 상속

IceButler_Server/src/main/java/com/example/icebutler_server/global/resolver/LoginResolver.java

Lines 71 to 75 in 078fa5b

	private Long getUserId(String accessToken) {
	Long userId = Long.valueOf(tokenUtils.getJwtContents(accessToken));
	userService.validateUser(userId);
	return userId;
	}

resolver가 실행되는 시점이 controller와 동일해서, userService를 거쳐 userRepository에 접근하는 것으로 생각했는데 옳은 방법인지 한 번만 같이 봐주세요 !

2. LoginResolver return값 UserId(Long type)로 변경

이유: LoginResolver에서 이제 유저 상태까지 점검 -> LoginStatus의 isLogin 컬럼 불필요
이점: 리졸버를 통해 바로 userId 반환 가능, 리졸버에서 isLogin 상태 변경 불필요
추가 작업 : LoginStatus 클래스 삭제 -> controller의 IsLogin 어노테이션 반환값 LoginStatus에서 Long으로 변경

만약 괜찮다면, 해당 브랜치에서 추가 작업 이어서 진행하겠습니다 의견 주세욥 !

• 참고
  

  IceButler_Server/src/main/java/com/example/icebutler_server/global/resolver/LoginStatus.java

  Lines 6 to 18 in 078fa5b

  	@Data
  	public class LoginStatus {
  	private Boolean isLogin;
  	private Long userId;
  	@Builder
  	public LoginStatus(Boolean isLogin, Long userId) {
  	this.isLogin = isLogin;
  	this.userId = userId;
  	}
  	public static LoginStatus getNotLoginStatus() {
  	return new LoginStatus(false, null);
  	}
  	}

[psyeon1120]

만약 괜찮다면, 해당 브랜치에서 추가 작업 이어서 진행하겠습니다 의견 주세욥 !

어떤 작업을 이어서 하시는건가요??

이해 완. 이어서 해주시면 좋을 것 같아요!

[psyeon1120]

😆

[sojungpp]

만약 괜찮다면, 해당 브랜치에서 추가 작업 이어서 진행하겠습니다 의견 주세욥 !

어떤 작업을 이어서 하시는건가요??

이해 완. 이어서 해주시면 좋을 것 같아요!

반영했습니다 ! 확인하고 머지해주세요 -!