Skip to content

Conversation

@1117mg
Copy link
Member

@1117mg 1117mg commented Aug 13, 2025

📌 PR 제목

📋 작업 내용

  • fix: 디스코드 알림 유형 제한

✅ 체크리스트

  • 테스트를 완료했나요?
  • 코드 컨벤션을 지켰나요?
  • 관련 문서를 업데이트 했나요?

@1117mg 1117mg self-assigned this Aug 13, 2025
@1117mg 1117mg added the bug Something isn't working label Aug 13, 2025
@1117mg 1117mg linked an issue Aug 13, 2025 that may be closed by this pull request
@claude
Copy link

claude bot commented Aug 13, 2025

Claude finished @1117mg's task —— View job


코드 리뷰 완료

변경사항 요약

이번 PR은 디스코드 알림 유형을 제한하기 위한 로깅 설정 개선을 포함합니다. 주요 변경사항은 application.yaml의 로깅 레벨 정리와 logback.xml에 EXCLUDE_DISCORD 마커 필터 추가입니다. 이를 통해 특정 로그가 디스코드로 전송되지 않도록 제어할 수 있게 되었습니다. 전반적으로 로깅 설정이 더 체계적으로 정리되었고, 운영환경과 개발환경 모두에 일관된 필터링 로직이 적용되었습니다.

코드 리뷰

주요 변경사항

파일 변경 내용 목적
application.yaml 로깅 레벨 재구성 및 운영용 설정 주석 추가 개발/운영 환경별 로깅 레벨 명확화
logback.xml EXCLUDE_DISCORD 마커 필터 추가 특정 로그의 디스코드 알림 제외

검토 결과

LGTM

변경사항이 명확하고 목적에 부합합니다. 디스코드 알림 제한을 위한 마커 기반 필터링 시스템이 올바르게 구현되었으며, 로깅 설정도 적절히 정리되었습니다.

할 일 목록

  • 변경된 파일들 읽기 및 분석 (application.yaml, logback.xml)
  • 한국어로 간단한 리뷰 요약 작성 (3-5문장)
  • 이슈/제안사항/질문 중심의 상세 코드 리뷰 댓글 제공
  • 완전한 리뷰로 GitHub 댓글 업데이트

@claude
Copy link

claude bot commented Aug 13, 2025

Claude finished @1117mg's task —— View job


코드 리뷰 완료

변경사항 요약

이번 PR은 디스코드 알림 유형을 제한하기 위한 로깅 설정 개선을 포함합니다. 주요 변경사항은 application.yaml의 로깅 레벨 정리와 logback.xml에 EXCLUDE_DISCORD 마커 필터 추가입니다. 이를 통해 특정 로그가 디스코드로 전송되지 않도록 제어할 수 있게 되었습니다. 전반적으로 로깅 설정이 더 체계적으로 정리되었고, 운영환경과 개발환경 모두에 일관된 필터링 로직이 적용되었습니다.

코드 리뷰

주요 변경사항

파일 변경 내용 목적
docker-compose.dev.yaml 이미지 태그 변경: dev-latestlatest 개발환경 이미지 태그 수정
docker-compose.prod.yaml 이미지 태그 변경: latestdev-latest 운영환경 이미지 태그 수정
application.yaml 로깅 레벨 재구성 및 운영용 설정 주석 추가 개발/운영 환경별 로깅 레벨 명확화
logback.xml EXCLUDE_DISCORD 마커 필터 추가 특정 로그의 디스코드 알림 제외

검토 결과

docker-compose.dev.yaml:27 - issue: 이미지 태그가 의도와 반대로 설정됨 - 개발환경에서 latest 태그를 사용하면 프로덕션 이미지를 가져올 수 있어 예상치 못한 동작 발생 가능

docker-compose.prod.yaml:27 - issue: 이미지 태그가 의도와 반대로 설정됨 - 운영환경에서 dev-latest 태그를 사용하면 개발용 이미지를 운영에 배포하게 되어 심각한 문제 발생 가능

할 일 목록

  • 변경된 파일들 읽기 및 분석 (docker-compose.dev.yaml, docker-compose.prod.yaml, application.yaml, logback.xml)
  • 한국어로 간단한 리뷰 요약 작성 (3-5문장)
  • 이슈/제안사항/질문 중심의 상세 코드 리뷰 댓글 제공
  • 완전한 리뷰로 GitHub 댓글 업데이트

@1117mg 1117mg merged commit 9ca1b56 into main Aug 13, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] 디스코드 알림 케이스 제한

2 participants