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

Logback 구성 및 환경별 설정 #235

Merged
merged 13 commits into from
Aug 5, 2024

Conversation

jminkkk
Copy link
Contributor

@jminkkk jminkkk commented Aug 1, 2024

⚡️ 관련 이슈

close #229

📍주요 변경 사항

  1. 로그백 설정 및 환경별 구성파일 작성
  2. 콘솔 및 파일용 appender 작성

🎸 기타

  1. 디스커션의 환경별 로깅 레벨 정하기을 참고해주세요.

기간

내일(8월 2일) 오전까지 부탁드립니다.

@jminkkk jminkkk added zap 리뷰 우선순위가 높은 사항 BE 백엔드 labels Aug 1, 2024
@jminkkk jminkkk added this to the 3차 스프린트 🐤 milestone Aug 1, 2024
@jminkkk jminkkk self-assigned this Aug 1, 2024
zangsu
zangsu previously approved these changes Aug 2, 2024
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.

몰리 로그 너무 멋져요 ~!
몰리가 했기에 이렇게 깔끔하게 코드가 작성될 수 있는거에요 굿굿 ~!
몇가지 제안사항 남겨놨으니 확인 부탁드립니다 ~

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.

몰리 정말 고생 많았어요.

의논할 게 있어 코멘트 남겼습니다 👍

HoeSeong123
HoeSeong123 previously approved these changes Aug 2, 2024
Copy link
Contributor

@HoeSeong123 HoeSeong123 left a comment

Choose a reason for hiding this comment

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

남기는 로그의종류가 너무 많아지면 정보를 찾기 힘들 수도 있겠다는 생각이 드네요.
그래도 이 부분은 일단 사용해보고 불편함을 느끼면 그 때 가서 다시 다같이 의논해보기로 하죠!!
고생하셨습니다.

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.

추가적으로 prod profile도 추가해줬으면 좋겠어요

@jminkkk jminkkk dismissed stale reviews from HoeSeong123 and zangsu via 890a951 August 2, 2024 08:53
@jminkkk jminkkk merged commit c42e27d into woowacourse-teams:dev/be Aug 5, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE 백엔드 zap 리뷰 우선순위가 높은 사항
Projects
Status: Weekend Done
Development

Successfully merging this pull request may close these issues.

5 participants