[REFACTOR] AOP 기반 로깅 세팅

[catomat0]

📌 PR 개요

• AOP 기반 로그 세팅

---

🔗 관련 이슈

• Closes [REFACTOR] 전역 로그 추가 #206

---

🛠 변경 사항

• AOP 클래스
• MDC패턴 필터

---

⚠️ 리뷰 시 참고 사항

---

✅ 체크리스트

• 로컬에서 정상 실행됨
• 로그 / 네이밍 정리
• main / develop 직접 커밋 아님

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: 275310d5-b1a7-487f-8a54-d425d3b898c9

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch refactor/#206-aop-logging

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[catomat0]

@CodeRabbit 로깅 제대로 먹히는지 체크해줘

[coderabbitai]

Tip

For best results, initiate chat on the files or code changes.

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Find AOP logging and MDC filter related files
fd -e java | xargs grep -l -i "aspect\|logging\|MDC\|filter" 2>/dev/null | head -30

# Also look for the specific files added/changed
git diff --name-only HEAD~1 2>/dev/null || git show --name-only HEAD 2>/dev/null | head -30

Length of output: 752

---

🏁 Script executed:

#!/bin/bash
set -e

echo "===== build.gradle ====="
nl -ba build.gradle | sed -n '1,400p' || true

echo
echo "===== LoggingAspect.java ====="
nl -ba src/main/java/com/deare/backend/global/aop/LoggingAspect.java | sed -n '1,400p' || true

echo
echo "===== MDCLoggingFilter.java ====="
nl -ba src/main/java/com/deare/backend/global/filter/MDCLoggingFilter.java | sed -n '1,400p' || true

echo
echo "===== SecurityConfig.java ====="
nl -ba src/main/java/com/deare/backend/global/config/SecurityConfig.java | sed -n '1,400p' || true

echo
echo "===== JwtAuthenticationFilter.java ====="
nl -ba src/main/java/com/deare/backend/global/auth/jwt/filter/JwtAuthenticationFilter.java | sed -n '1,400p' || true

echo
echo "===== P6SpySqlLogFormatter.java ====="
nl -ba src/main/java/com/deare/backend/global/config/p6spy/P6SpySqlLogFormatter.java | sed -n '1,400p' || true

echo
echo "===== logback-spring.xml ====="
nl -ba src/main/resources/logback-spring.xml | sed -n '1,400p' || true

Length of output: 529

---

🏁 Script executed:

#!/bin/bash

echo "===== build.gradle ====="
cat -n build.gradle 2>/dev/null || true

echo
echo "===== LoggingAspect.java ====="
cat -n src/main/java/com/deare/backend/global/aop/LoggingAspect.java 2>/dev/null || true

echo
echo "===== MDCLoggingFilter.java ====="
cat -n src/main/java/com/deare/backend/global/filter/MDCLoggingFilter.java 2>/dev/null || true

echo
echo "===== SecurityConfig.java ====="
cat -n src/main/java/com/deare/backend/global/config/SecurityConfig.java 2>/dev/null || true

echo
echo "===== JwtAuthenticationFilter.java ====="
cat -n src/main/java/com/deare/backend/global/auth/jwt/filter/JwtAuthenticationFilter.java 2>/dev/null || true

echo
echo "===== P6SpySqlLogFormatter.java ====="
cat -n src/main/java/com/deare/backend/global/config/p6spy/P6SpySqlLogFormatter.java 2>/dev/null || true

echo
echo "===== logback-spring.xml ====="
cat -n src/main/resources/logback-spring.xml 2>/dev/null || true

Length of output: 23953

---

@catomat0 확인 결과 요약입니다.

무엇이 연결되어 있는지

• AOP 활성화: build.gradle Line 29에 spring-boot-starter-aop 추가됨. LoggingAspect는 @Aspect/@component로 등록되어 서비스 계층 메서드 호출 시 INFO 로그와 실행시간(ms)을 남깁니다. (LoggingAspect.java Line 10–27)
• MDC + 패턴: logback-spring.xml에 [%X{requestId}] [userId=%X{userId}] [%X{method} %X{uri}] 패턴이 선언되어 있고, MDCLoggingFilter가 requestId/method/uri/userId를 세팅합니다. (MDCLoggingFilter.java Line 31–41, logback-spring.xml Line 7–14)
• 인증 연계: JwtAuthenticationFilter가 principal을 Long userId로 세팅합니다. (JwtAuthenticationFilter.java Line 44–59)

체크 포인트와 개선 제안

1. MDC 필터 실행 순서

• 현재 MDCLoggingFilter는 @Component만 되어 있고 SecurityFilterChain에 명시적으로 추가되진 않았습니다. userId를 정확히 찍으려면 JwtAuthenticationFilter가 SecurityContext를 채운 “이후”에 MDC 필터가 동작해야 안전합니다.
• 제안: SecurityConfig에서 MDCLoggingFilter를 JwtAuthenticationFilter 뒤에 추가해 순서를 고정하세요.

예시 패치
Java
// SecurityConfig.java
import com.deare.backend.global.filter.MDCLoggingFilter;
import org.springframework.security.web.context.SecurityContextHolderFilter; // 필요 시

@Configuration
@EnableWebSecurity
@RequiredArgsConstructor
public class SecurityConfig {

private final JwtAuthenticationFilter jwtAuthenticationFilter;
private final MDCLoggingFilter mdcLoggingFilter; // 주입

`@Bean`
public SecurityFilterChain filterChain(HttpSecurity http) throws Exception {
    // ... 기존 설정 ...

    http.addFilterBefore(jwtAuthenticationFilter, UsernamePasswordAuthenticationFilter.class);

    // Jwt 인증 후 MDC 세팅되도록 순서 보장
    http.addFilterAfter(mdcLoggingFilter, JwtAuthenticationFilter.class);

    return http.build();
}

}

2. 예외 로그 스택트레이스

• 현재 LoggingAspect의 에러 로그는 e.getMessage()만 출력해 스택트레이스가 남지 않습니다. 운영 문제 분석을 위해 Throwable을 함께 넘기도록 수정 권장합니다.

예시 패치
Java
// LoggingAspect.java (catch 블록)
} catch (Exception e) {
long elapsed = System.currentTimeMillis() - start;
log.error("[{}] {} - {}ms - 예외", className, methodName, elapsed, e); // 스택트레이스 포함
throw e;
}

3. principal 파싱 보강(옵션)

• 현 구현은 principal이 Long일 때만 userId를 기록합니다. 다른 인증 흐름(예: OAuth2)에서도 이름 기반 식별이 가능하도록 fallback을 두면 좋습니다.

예시 패치
Java
Authentication auth = SecurityContextHolder.getContext().getAuthentication();
if (auth != null) {
Object p = auth.getPrincipal();
if (p instanceof Long userId) {
MDC.put(USER_ID, String.valueOf(userId));
} else if (p instanceof org.springframework.security.core.userdetails.UserDetails ud) {
MDC.put(USER_ID, ud.getUsername());
} else if (auth.getName() != null) {
MDC.put(USER_ID, auth.getName());
} else {
MDC.put(USER_ID, "anonymous");
}
} else {
MDC.put(USER_ID, "anonymous");
}

4. P6Spy 포매터 활성화(확인 필요)

• 커스텀 P6SpySqlLogFormatter 클래스는 존재하지만, 활성화를 위한 설정(application*.yml/ properties)이 보이지 않습니다. 사용 중인 스타터( gavlyukovskiy p6spy-spring-boot-starter ) 기준으로 커스텀 포맷터를 지정해야 적용됩니다.
• 예시(application.yml; 실제 사용하는 설정 키를 프로젝트 표준에 맞춰 적용해 주세요)

YAML
decorator:
datasource:
p6spy:
enable-logging: true
logging: slf4j
# 커스텀 포맷터
log-format: com.deare.backend.global.config.p6spy.P6SpySqlLogFormatter

또는 순수 p6spy 설정을 쓸 경우:
YAML
p6spy:
config:
appender: com.p6spy.engine.spy.appender.Slf4JLogger
logMessageFormat: com.deare.backend.global.config.p6spy.P6SpySqlLogFormatter

간단 검증 방법(로컬)

• dev 프로필로 애플리케이션 기동 후, 아무 API 1회 호출
• 기대 로그 예시
  2026-04-08 12:34:56.789 [http-nio-8080-exec-1] INFO [a1b2c3d4] [userId=123] [GET /api/v1/test/ping] com.deare...LoggingAspect - [LetterServiceImpl] someMethod - 12ms
• userId가 anonymous로만 찍히면 필터 순서가 잘못된 것입니다. 위 1) 패치 반영 후 재확인해 주세요.

결론

• AOP 타이밍 로그와 MDC 패턴은 준비되어 있어 “기본 출력”은 정상입니다.
• 정확한 userId 포함과 예외 분석 품질을 위해
  • MDCLoggingFilter를 JwtAuthenticationFilter 뒤에 명시 추가
  • LoggingAspect 에러 로그에 Throwable 포함
  • (선택) principal 파싱 보강
  • (선택) P6Spy 포매터 활성화 설정
    을 반영하면 좋겠습니다. 원하시면 위 변경으로 PR 보강 계획을 잡아드릴게요.