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.

Sign up for GitHub

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

Jump to bottom

초 기본적인 로깅 구현 #176

Merged
merged 6 commits into from
Jul 29, 2023
Merged

초 기본적인 로깅 구현 #176

merged 6 commits into from
Jul 29, 2023

Conversation

jjongwa
Copy link
Member

@jjongwa jjongwa commented Jul 29, 2023

📄 Summary

#115

초초 기본적인 로깅 파일 생성 로직 구현해 놨습니다.

  • warn과 error 레벨의 로그만 파일에 기록

  • 위치: backend/log 디렉토리에 warn과 error 디렉토리 만들어지고 거기에 각각 파일 생성됨

  • 파일 네이밍: warn-2023-07-29.log 형식으로 파일 생성

  • 용량 제한: 각 파일당 100MB, 디렉토리 당 총 파일 개수는 10개 제한 (파일이 10개일 경우 가장 오래된 파일 삭제)

🙋🏻 More

저희가 현재 hanglog-dev 에 백엔드 코드 , nginx, 사진 파일까지 올라가고 있는 상황이라 로그 파일의 용량 제한을 warn, error 각각 1GB, 총 2GB로 빡세게 제한을 걸어 놨습니다. 각각 어느정도로 용량을 할당해야 할 지 논의가 필요할 듯 합니다. 역시 사진은 S3로 옮기는게..

@jjongwa jjongwa self-assigned this Jul 29, 2023
@jjongwa jjongwa linked an issue Jul 29, 2023 that may be closed by this pull request
로깅 구현 #115
Closed
3 tasks
@github-actions
Copy link

github-actions bot commented Jul 29, 2023 •
edited
Loading

📝 Jacoco Test Coverage

Total Project Coverage 81.8% 🍏
File Coverage [47.27%]
GlobalExceptionHandler.java 47.27% ❌

hgo641
hgo641 approved these changes Jul 29, 2023
View reviewed changes
Copy link
Collaborator

@hgo641 hgo641 left a comment

Choose a reason for hiding this comment

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

디노고생고생하셨습니다~

waterricecake
waterricecake approved these changes Jul 29, 2023
View reviewed changes
Copy link
Collaborator

@waterricecake waterricecake left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

mcodnjs
mcodnjs approved these changes Jul 29, 2023
View reviewed changes
Copy link
Collaborator

@mcodnjs mcodnjs left a comment

Choose a reason for hiding this comment

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

조은데?

@jjongwa jjongwa merged commit a67eb48 into develop Jul 29, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mcodnjs mcodnjs mcodnjs approved these changes

@hgo641 hgo641 hgo641 approved these changes

@waterricecake waterricecake waterricecake approved these changes

Assignees

@jjongwa jjongwa

Labels
BE ✨ Feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

로깅 구현
4 participants
@jjongwa @mcodnjs @hgo641 @waterricecake