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

[BE] fix: logback으로 남기는 로그 통합 (#888) #889

Draft
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

seokjin8678
Copy link
Collaborator

@seokjin8678 seokjin8678 commented Apr 20, 2024

📌 관련 이슈

✨ PR 세부 내용

이슈 내용에서 작성한 내용과 같이 logback으로 남기는 로그 구분을 하나로 통합했습니다.

springProfile 별로 나뉘어진 중복된 로그 설정을 하나로 통합했고, 로그는 /log/{level}/ 디렉터리에 남겨집니다.

추가로 GlobalExceptionHandler에서 사용하던 ErrorLogger@Slf4j 사용하도록 변경했습니다.

@seokjin8678 seokjin8678 added BE 백엔드에 관련된 작업 🏢 인프라 인프라에 관련된 작업 ⚙️ 리팩터링 리팩터링에 관련된 작업 labels Apr 20, 2024
@seokjin8678 seokjin8678 self-assigned this Apr 20, 2024
@github-actions github-actions bot requested review from BGuga, carsago and xxeol2 April 20, 2024 17:46
Copy link

github-actions bot commented Apr 20, 2024

Test Results

200 files  200 suites   25s ⏱️
665 tests 665 ✅ 0 💤 0 ❌
678 runs  678 ✅ 0 💤 0 ❌

Results for commit 7129cee.

♻️ This comment has been updated with latest results.

@seokjin8678 seokjin8678 added 🛠 수정 수정에 관련된 작업 and removed ⚙️ 리팩터링 리팩터링에 관련된 작업 labels Apr 20, 2024
@seokjin8678 seokjin8678 marked this pull request as draft May 30, 2024 01:17
@seokjin8678
Copy link
Collaborator Author

해당 사항 merge 하면 인프라 설정을 변경해야 하므로 우선 draft 처리합니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE 백엔드에 관련된 작업 🏢 인프라 인프라에 관련된 작업 🛠 수정 수정에 관련된 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BE] logback에 남기는 로그 구분을 하나로 통합한다.
3 participants