-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
#338 [feat] Observability 개선 #339
Conversation
좋습니다-! P6spy를 ec2 t2micro 환경에서 사용하면 메모리 부족도 있을 것 같아, 환경 분리도 고려해보면 좋을 것 같습니다~! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
좋습니다
@sohyundoh
혹시 p6spy 관련 메모리릭 같은 이슈가 따로 있을까요??? |
@KWY0218 |
✒️ 관련 이슈번호
Key Changes 🔑
1. MDC Filter 추가
로그가 섞여도 특정 스레드의 행동을 파악할 수 있도록 MDC 정보를 추가했습니다.
metric 시 타 스레드로 동작하기 때문에 mdc 내용이 전파되지 않아, TaskDecorator를 추가해 async 시에도 mdc 내용이 전파되도록 구현했습니다.
2. 요청 및 응답 값 로깅
aop를 통해 요청에 대한 요청 값과 응답 값을 로깅하도록 구현했습니다.
controller 에 들어온 아규먼트를 추출해 요청 값을 로깅했고, returnValue를 통해 응답 값을 로깅했습니다.
비즈니스 중 에러가 발생했을 때 controllerAdvice에서 응답 값을 반환하기 때문에 controllerAdvice에 대한 pointCut을 만들어 에러 시 응답 값을 로깅했습니다.
3. p6spy dependency 추가
sql에 대한 로깅을 하고 싶어 p6spy 라이브러리를 추가했습니다.
p6spy의 메시지 툴도 괜찮은 것 같다고 생각해 별도의 커스텀은 하지 않았습니다..!
4. rolling file appender로 수정
요일 단위로 로그를 수집하도록 file appender -> rolling file appender로 수정했습니다.
To Reviewers 📢
as-is
to-be