Skip to content

Conversation

@seoyeonjin
Copy link
Member

Related Issue 🚀

Work Description ✏️

  • 필요없는 로그 (ex. db 연결 로그) 가 뜨지 않도록 로그 레벨을 변경했습니다.
  • AWS SDK 관련 패키지 로그 레벨은 INFO로 설정했습니다.
  • 애플리케이션 패키지(com.sopt.push)는 디버깅 편의를 위해 DEBUG로 유지했습니다.

PR Point 📸

@seoyeonjin seoyeonjin self-assigned this Dec 31, 2025
@seoyeonjin seoyeonjin added the 📝 chore .github 아래에 존재하는 yml(ex. release-drafter)을 수정하는 경우 ( + 인프라와 관련되지 않은 로직일 경우) + 패키지 구조 변경, 파일 이동 등 label Dec 31, 2025
@coderabbitai
Copy link

coderabbitai bot commented Dec 31, 2025

Summary by CodeRabbit

릴리스 노트

  • Chores
    • 로그 설정을 추가/개선하여 콘솔 출력 형식(타임스탬프, 레벨, 스레드, 로거, 메시지 포함)을 표준화했습니다.
    • 특정 내부 라이브러리의 로깅 레벨을 INFO로 조정하고, 애플리케이션용 로거는 DEBUG로 세부 로깅을 허용해 문제 추적을 용이하게 했습니다.

✏️ Tip: You can customize this high-level summary in your review settings.

Walkthrough

새로운 로깅 설정 파일 src/main/resources/logback.xml을 추가했습니다. AWS SDK 관련 패키지들의 로그 레벨을 INFO로, com.sopt.push는 DEBUG로 설정하고 콘솔(Appender STDOUT) 출력 포맷을 정의합니다.

Changes

Cohort / File(s) 변경 내용
로깅 설정 추가
src/main/resources/logback.xml
새 파일 생성: ConsoleAppender(STDOUT) 정의, 출력 패턴(타임스탬프, 레벨, 스레드, 로거, 메시지), AWS SDK 패키지(software.amazon.awssdk, software.amazon.awssdk.*.request, *.http, *.core)를 INFO로, com.sopt.push를 DEBUG로 설정.

Sequence Diagram(s)

(생략 — 변경이 구성 파일 추가로, 다중 컴포넌트 제어 흐름 변경을 수반하지 않음)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 분

Suggested labels

size/XS

Suggested reviewers

  • JungYoonShin
  • sung-silver

Poem

🐇 로그밭에 살짝 바람이 불어와,
INFO는 낮게 흩어지고 DEBUG는 춤추네.
콘솔에 빛나는 패턴 한 줄로,
소음은 줄이고 필요한 소리만 남겼지.
깡총 — 정리된 로그로 마음도 가벼워!

Pre-merge checks

✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 실제 변경사항(logback.xml을 통한 로그 레벨 설정)과 일치하며 명확하고 간결합니다.
Description check ✅ Passed PR 설명이 필수 섹션(Related Issue, Work Description)을 포함하고 있으며, 로그 레벨 변경의 구체적인 내용을 상세히 설명하고 있습니다.
Linked Issues check ✅ Passed PR의 변경사항(AWS SDK 관련 패키지 로그 레벨을 INFO로 설정, com.sopt.push는 DEBUG로 유지)이 연결된 이슈 #26의 요구사항(불필요한 로그 감소를 위한 로그 레벨 조정)을 충족합니다.
Out of Scope Changes check ✅ Passed PR의 모든 변경사항(logback.xml 설정 추가)이 로그 레벨 설정이라는 정의된 범위 내에 있으며, 애플리케이션 로직 변경은 없습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 78a4d52 and afbfd65.

📒 Files selected for processing (1)
  • src/main/resources/logback.xml
🚧 Files skipped from review as they are similar to previous changes (1)
  • src/main/resources/logback.xml

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🧹 Nitpick comments (1)
src/main/resources/logback.xml (1)

2-5: AWS SDK 하위 패키지 로거 설정의 중복 가능성

logback의 로거 계층 구조에서 software.amazon.awssdk를 INFO로 설정하면 그 하위 패키지들(.request, .http, .core)도 자동으로 INFO 레벨을 상속받습니다. 명시적인 하위 패키지 설정은 의도를 명확히 하는 장점은 있지만, 유지보수 시 일관성 유지에 주의가 필요합니다.

특정 하위 패키지만 다른 레벨로 설정할 계획이 없다면 2번 라인만 유지하는 것도 고려해보세요.

🔎 간소화된 설정 예시
     <logger name="software.amazon.awssdk" level="INFO"/>
-    <logger name="software.amazon.awssdk.request" level="INFO"/>
-    <logger name="software.amazon.awssdk.http" level="INFO"/>
-    <logger name="software.amazon.awssdk.core" level="INFO"/>
📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 928ea0e and 78a4d52.

📒 Files selected for processing (1)
  • src/main/resources/logback.xml
🔇 Additional comments (1)
src/main/resources/logback.xml (1)

7-7: 애플리케이션 로거 설정 적절함

com.sopt.push 패키지를 DEBUG 레벨로 유지하여 디버깅 편의성을 확보한 설정이 PR 목적에 부합합니다.

Copy link
Member

@JungYoonShin JungYoonShin 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

@sung-silver sung-silver left a comment

Choose a reason for hiding this comment

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

good~ 고생하셨어용

@seoyeonjin seoyeonjin merged commit 1f4655d into develop Jan 9, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📝 chore .github 아래에 존재하는 yml(ex. release-drafter)을 수정하는 경우 ( + 인프라와 관련되지 않은 로직일 경우) + 패키지 구조 변경, 파일 이동 등 size/S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CHORE] 로그 레벨 변경

4 participants