[TNT-28] refactor: 변수, 로그 개행 수정

Author: fakerdeft

Diff for: src/main/java/com/tnt/application/auth/SessionService.java

@@ -28,13 +28,14 @@ public class SessionService {
 
 	public String authenticate(HttpServletRequest request) {
 		String authHeader = request.getHeader(AUTHORIZATION_HEADER);
-		String sessionId;
 
 		if (isBlank(authHeader) || !authHeader.startsWith(SESSION_ID_PREFIX)) {
 			log.error("Authorization 헤더가 존재하지 않거나 올바르지 않은 형식입니다.");
+
 			throw new UnauthorizedException("인가 세션이 존재하지 않습니다.");
 		}
-		sessionId = authHeader.substring(SESSION_ID_PREFIX.length());
+
+		String sessionId = authHeader.substring(SESSION_ID_PREFIX.length());
 
 		requireNonNull(redisTemplate.opsForValue().get(sessionId), "세션 스토리지에 세션이 존재하지 않습니다.");
 

Diff for: src/main/java/com/tnt/global/auth/SessionAuthenticationFilter.java

@@ -39,8 +39,10 @@ protected void doFilterInternal(HttpServletRequest request, HttpServletResponse
 
 		log.info("들어온 요청 - URI: {}, Query: {}, Method: {}", requestUri, queryString != null ? queryString : "쿼리 스트링 없음",
 			request.getMethod());
+
 		if (isAllowedUri(requestUri)) {
 			log.info("{} 허용 URI. 세션 유효성 검사 스킵.", requestUri);
+
 			filterChain.doFilter(request, response);
 			return;
 		}
@@ -49,6 +51,7 @@ protected void doFilterInternal(HttpServletRequest request, HttpServletResponse
 			checkSessionAndAuthentication(request);
 		} catch (RuntimeException e) {
 			log.error("인증 처리 중 에러 발생: ", e);
+
 			handleUnauthorizedException(response, e);
 			return;
 		}
@@ -65,6 +68,7 @@ private boolean isAllowedUri(String requestUri) {
 				break;
 			}
 		}
+
 		log.info("URI {} is {}allowed", requestUri, allowed ? "" : "not ");
 
 		return allowed;
@@ -79,6 +83,7 @@ private void checkSessionAndAuthentication(HttpServletRequest request) {
 	private void handleUnauthorizedException(HttpServletResponse response, RuntimeException exception) throws
 		IOException {
 		log.error("인증 실패: ", exception);
+
 		response.setStatus(HttpServletResponse.SC_UNAUTHORIZED);
 		response.setContentType("application/json;charset=UTF-8");
 		response.getWriter().write(exception.getMessage());
@@ -95,6 +100,7 @@ private void saveAuthentication(Long sessionId) {
 			authoritiesMapper.mapAuthorities(userDetails.getAuthorities()));
 
 		SecurityContextHolder.getContext().setAuthentication(authentication);
+
 		log.info("시큐리티 컨텍스트에 인증 정보 저장 완료 - SessionId: {}", sessionId);
 	}
 }

Diff for: src/main/java/com/tnt/global/error/handler/GlobalExceptionHandler.java

@@ -39,6 +39,7 @@ public class GlobalExceptionHandler {
 	@ExceptionHandler(MissingServletRequestParameterException.class)
 	protected ErrorResponse handleMissingServletRequestParameter(MissingServletRequestParameterException exception) {
 		String errorMessage = String.format("필수 파라미터 '%s'가 누락되었습니다.", exception.getParameterName());
+
 		log.warn("Required request parameter is missing: {}", exception.getParameterName());
 
 		return new ErrorResponse(errorMessage);
@@ -48,12 +49,11 @@ protected ErrorResponse handleMissingServletRequestParameter(MissingServletReque
 	@ResponseStatus(HttpStatus.BAD_REQUEST)
 	@ExceptionHandler(MethodArgumentTypeMismatchException.class)
 	protected ErrorResponse handleMethodArgumentTypeMismatch(MethodArgumentTypeMismatchException exception) {
-		String errorMessage;
-		log.warn("Type mismatch for parameter: {}. Required type: {}", exception.getName(),
+		String errorMessage = String.format("파라미터 '%s'의 형식이 올바르지 않습니다. 예상 타입: %s", exception.getName(),
 			requireNonNull(exception.getRequiredType()).getSimpleName());
 
-		errorMessage = String.format("파라미터 '%s'의 형식이 올바르지 않습니다. 예상 타입: %s", exception.getName(),
-			exception.getRequiredType().getSimpleName());
+		log.warn("Type mismatch for parameter: {}. Required type: {}", exception.getName(),
+			requireNonNull(exception.getRequiredType()).getSimpleName());
 
 		return new ErrorResponse(errorMessage);
 	}
@@ -62,15 +62,14 @@ protected ErrorResponse handleMethodArgumentTypeMismatch(MethodArgumentTypeMisma
 	@ResponseStatus(HttpStatus.BAD_REQUEST)
 	@ExceptionHandler(ConstraintViolationException.class)
 	protected ErrorResponse handleConstraintViolationException(ConstraintViolationException exception) {
-		log.warn("Constraint violation: {}", exception.getMessage());
-
 		List<String> errors = exception.getConstraintViolations()
 			.stream()
 			.map(violation -> violation.getPropertyPath() + ": " + violation.getMessage())
 			.toList();
-
 		String errorMessage = String.join(", ", errors);
 
+		log.warn("Constraint violation: {}", exception.getMessage());
+
 		return new ErrorResponse("입력값이 유효하지 않습니다: " + errorMessage);
 	}
 
@@ -120,6 +119,7 @@ protected ErrorResponse handleRuntimeException(RuntimeException exception) {
 
 		String errorKeyInfo = String.format(ERROR_KEY_FORMAT, sb);
 		String exceptionTypeInfo = String.format(EXCEPTION_CLASS_TYPE_MESSAGE_FORMANT, exception.getClass());
+
 		log.error("{}{}{}", exception.getMessage(), errorKeyInfo, exceptionTypeInfo);
 
 		return new ErrorResponse(DEFAULT_ERROR_MESSAGE + errorKeyInfo);