Skip to content
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

Open
wants to merge 1 commit into
base: dev/be
Choose a base branch
from

Conversation

HoeSeong123
Copy link
Contributor

⚡️ 관련 이슈

close #989

📍주요 변경 사항

로그에 헤더가 너무 많아서 필요한 정보만 포함되도록 수정하였습니다.
실행중인 서버의 도커 컨테이너 정보가 response 로그에 포함되도록 수정하였습니다.
image
적혀있는 정보는 실행중인 도커의 컨테이너 ID 입니다.

무중단 배포를 도입하면서 현재 어떤 컨테이너가 실행중인지 파악하기 어렵다는 문제가 있었는데 이 로그를 통해 해결할 수 있을 것 같습니당.

🎸기타

현재 request 로그에는 origin, host, content-type을 추가하였고, response 로그에는 docker-hostname만 추가되도록 하였습니다.
더 추가되었으면 하는 헤더 정보가 있다면 얘기해주세용.

🍗 PR 첫 리뷰 마감 기한

12/26 23:59

@HoeSeong123 HoeSeong123 added BE 백엔드 infrastructure 인프라 작업 labels Dec 23, 2024
@HoeSeong123 HoeSeong123 added this to the 7차 스프린트 💭 milestone Dec 23, 2024
@HoeSeong123 HoeSeong123 self-assigned this Dec 23, 2024
Copy link
Contributor

@zeus6768 zeus6768 left a 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");
Copy link
Contributor

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");
Copy link
Contributor

Choose a reason for hiding this comment

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

여기도 static final 어떨까요~!

Copy link
Contributor

@kyum-q kyum-q left a comment

Choose a reason for hiding this comment

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

롱초 체거에요~~~~~!!!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE 백엔드 infrastructure 인프라 작업
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

3 participants