-
Notifications
You must be signed in to change notification settings - Fork 8
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
로그에 헤더 정보 수정 및 컨테이너 정보 추가 #990
base: dev/be
Are you sure you want to change the base?
Conversation
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.
🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥
@@ -65,20 +67,27 @@ private void logResponse(HttpServletRequest request, HttpServletResponse respons | |||
} | |||
|
|||
private String getHeaderAsJson(ContentCachingRequestWrapper requestWrapper) { | |||
Set<String> requiredHeaders = Set.of("origin", "host", "content-type"); |
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.
메서드 실행할 때마다 객체를 생성하는 대신, static final
로 선언하면 어떨까요??
return convertMapToJson(headersMap); | ||
} | ||
|
||
private String getHeaderAsJson(ContentCachingResponseWrapper responseWrapper) { | ||
Set<String> requiredHeaders = Set.of("docker-hostname"); |
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.
여기도 static final
어떨까요~!
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.
롱초 체거에요~~~~~!!!!
⚡️ 관련 이슈
close #989
📍주요 변경 사항
로그에 헤더가 너무 많아서 필요한 정보만 포함되도록 수정하였습니다.
실행중인 서버의 도커 컨테이너 정보가 response 로그에 포함되도록 수정하였습니다.
적혀있는 정보는 실행중인 도커의
컨테이너 ID
입니다.🎸기타
🍗 PR 첫 리뷰 마감 기한
12/26 23:59