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

Implement Logging Filter #88

Merged
merged 5 commits into from
Sep 26, 2023
Merged

Implement Logging Filter #88

merged 5 commits into from
Sep 26, 2023

Conversation

jinlee1703
Copy link
Member

PR Type

  • 기능 추가
  • 기능 삭제
  • 버그 수정
  • 테스트 코드 작성
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트
  • 문서 작성

Motivation

WAS 서버의 로그를 저장하기 위한 Logging 구현 및 Logging Filter 구현

Problem Solving

  • log 관련 .xml 파일 정의
  • LoggingFilter 구현

To Reviewer

  • 우리 서비스에 맞게 잘 작성해보고 싶었으나 아직 많이 부족할 것 같습니다...
  • 추후 수정 및 보완이 필요하지 않을까 싶습니다.
  • 현재는 API 요청 시 request-body 출력, response-body 출력 등 API 단위에 대해서 로깅이 이루어지고 있습니다.

@jinlee1703 jinlee1703 requested a review from inh2613 September 25, 2023 17:39
@jinlee1703 jinlee1703 self-assigned this Sep 25, 2023
Copy link
Member

@inh2613 inh2613 left a comment

Choose a reason for hiding this comment

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

p6:
request header, request body, response body 이렇게 logging 하고 있는 것 맞을까요?
여쭤보고 싶어도 모르는 것이 많아서 여쭤볼게 적은 것 같습니다..
수고하셨습니다~

@jinlee1703
Copy link
Member Author

p6: request header, request body, response body 이렇게 logging 하고 있는 것 맞을까요? 여쭤보고 싶어도 모르는 것이 많아서 여쭤볼게 적은 것 같습니다.. 수고하셨습니다~

맞습니다! 저도 조금 더 이해하기 쉽게 코드를 작성하고 싶었는데, 시간이 부족해서 아쉽습니다 ㅠㅠ 나중에 수정할 기회가 생긴다면 보다 깔끔하게 구현해보고 싶습니다!

@jinlee1703 jinlee1703 merged commit 52d2c72 into dev Sep 26, 2023
2 checks passed
@jinlee1703 jinlee1703 deleted the feature/GH-273-logging branch September 26, 2023 06:20
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