Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[#35] Refactor : 로그 설정 변경 #45

Merged
merged 3 commits into from
Aug 21, 2023
Merged

Conversation

gombasan
Copy link
Member

작업 사항

  • 운영 환경 구축 #35 -> 로그 설정 내용이 해당 이슈와 어울린다고 생각하여 해당 이슈로 설정했습니다!!

작업 내용

refactor : logback 설정

  • 기존 로그파일 저장에서 ERROR level 은 따로 저장하여 관리하도록 수정했습니다.
  • 많은 로그 정보내에서 error 를 찾는것보다 파일을 따로 지정하여 관리함으로 운영상 더 편할거라고 생각하여 적용해봤습니다

refactor : 로그 출력 FORM 수정

  • 기존 로그 출력 부분에서 서버 에러일 경우 단순하게 서버에러입니다 와 메서드 이름만을 출력하니 로그 내용울 분석하는것이 거의 불가능하다 생각하여, 에러의 메세지를 추가로 더 출력하도록 수정하였습니다.
  • 최근 서버 공격을 받고있다는 알람 및 로그가 확인되어 클라이언트의 ip 를 로그로 출력하도록 설정하였습니다.

config : log 디렉토리 volumes 설정 추가

  • log 디렉토리를 도커 컨테이너 내부가 아닌 서버에서 확인 할 수 있도록 volumes 설정을 추가하였습니다.

리뷰 포인트

  • 로그에서 error 내용만 따로 뽑아서 관리하도록 설정해봤는데 괜찮은 방법인지 궁금합니다~~괜히 파일만 늘어나는게 아닌가 싶네요!

- 기존 로그파일 저장에서 ERROR level 은 따로 저장하여 관리하도록 수정했습니다.
- 많은 로그 정보내에서 error 를 찾는것보다 파일을 따로 지정하여 관리함으로 운영상 더 편할거라고 생각하여 적용해봤습니다
- 기존 로그 출력 부분에서 서버 에러일 경우 단순하게 `서버에러입니다` 와 메서드 이름만을 출력하니 로그 내용울 분석하는것이 거의 불가능하다 생각하여, 에러의 메세지를 추가로 더 출력하도록 수정하였습니다.
- 최근 서버 공격을 받고있다는 알람 및 로그가 확인되어 클라이언트의 ip 를 로그로 출력하도록 설정하였습니다.
- log 디렉토리를 도커 컨테이너 내부가 아닌 서버에서 확인 할 수 있도록 volumes 설정을 추가하였습니다.
@gombasan gombasan added 🔨 Refactor 코드 리팩토링 🔁 Devops 운영 관련 labels Aug 19, 2023
@gombasan gombasan self-assigned this Aug 19, 2023
@gombasan gombasan linked an issue Aug 19, 2023 that may be closed by this pull request
@github-actions
Copy link

테스트 커버리지 리포트입니다

File Coverage [79.13%]
MyLogger.java 79.13%
Total Project Coverage 78.21%

@gombasan
Copy link
Member Author

@JoosungKwon
머지 하겠습니다~~!!!

@gombasan gombasan merged commit 79cad0c into develop Aug 21, 2023
@JoosungKwon
Copy link
Collaborator

로그를 파일로 관리한다니 아주 좋습니다!! 나중에 ELK 스택같은걸 적용해보고 싶네요! 고생하셨습니다. 안그래도 너무 많아서 보기 힘들었으니까요! 시간나면 모니터링 시스템을 구축해보고자 합니다~!!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
🔁 Devops 운영 관련 🔨 Refactor 코드 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

운영 환경 구축
2 participants