Skip to content

Chore/243 로깅 시스템 추가 - #251

Merged
jaylovegood merged 3 commits into
devfrom
chore/243-로깅-시스템-추가
Jun 10, 2026

Hidden character warning

The head ref may contain hidden characters: "chore/243-\ub85c\uae45-\uc2dc\uc2a4\ud15c-\ucd94\uac00"
Merged

jaylovegood merged 3 commits into
devfrom
chore/243-로깅-시스템-추가

Conversation

@jaylovegood

Copy link
Copy Markdown
Member

✨ Feature PR (to dev)

🧪 로컬 테스트 여부 (작업자 체크)

  • 로컬에서 Swagger 혹은 테스트 코드로 동작을 확인했습니다.
  • API 변경이 있다면 src/main/resources/static/docs/openapi.yaml을 함께 갱신했습니다.
  • /docs/swagger-ui.html에서 변경 API가 의도대로 보이는지 확인했습니다.

📄 documentation 최신화 여부

변경사항과 관련된 API의 swagger documentation이 실제 동작과 일치하는지 확인합니다.

  • (작업자) 확인하였습니다.
  • (리뷰어) 확인하였습니다.

📌 작업 내용(what & why)

RequestId 추적

  • 요청마다 UUID를 생성해 MDC에 주입하는 RequestIdFilter 추가
  • 모든 로그에 [requestId] 포함, 응답 헤더 X-Request-Id로도 노출

LoggingAspect

  • 서비스 레이어에서 발생하는 예외를 AOP로 일괄 수집
  • DomainException 중 4xx (유저가 잘못된 값을 줘서 예외가 발생하는 게 정상인 경우) → 콜스택 없이 WARN 한 줄만 출력
  • 5xx DomainException 및 그 외 → ERROR + 콜스택 출력

logback 설정

  • 비동기 appender(AsyncAppender) 적용으로 로깅 I/O가 요청 처리에 영향을 주지 않도록 구성
  • prod 프로파일: 큐 80% 초과 시 DEBUG/INFO 자동 드롭, 최소 레벨 INFO (메모리 영향 최소화하기 위함)
  • 비prod 프로파일(dev, local 등): 로그 드롭 없음, 최소 레벨 DEBUG
  • prod에서 Spring JDBC 쿼리 로그 레벨 INFO로 고정

🔎 영향 범위

  • API 스펙 변경

📡 API 변경사항 (있다면)

모든 응답 Header에 X-Request-Id 추가

🔥 관련 이슈

@jaylovegood jaylovegood linked an issue May 19, 2026 that may be closed by this pull request
1 of 4 tasks

@uykhc uykhc left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

바쁘신데 작업하느라 수고하셨습니다!

현재 매 요청마다 새 UUID를 발급해서 기존 X-Request-Id를 덮어쓴다고 합니다.
API Gateway/Ingress가 이미 request id를 넣는 구조면, 서비스 로그와 외부 로그의 상관관계가 끊어질 수 있다고 합니다.
이 부분 한번 확인해주시면 감사하겠습니다.

@jaylovegood

Copy link
Copy Markdown
Member Author

저희의 경우 pod 로그를 직접 보는 거라서 크게 상관이 없을 것 같다는 결론을 내렸습니다..!!! 리뷰 감사합니다!

@jaylovegood
jaylovegood merged commit 85109ed into dev Jun 10, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

로깅 시스템 추가

2 participants