Fix/emer

[pokerbearkr]

PR 생성 시 아래 항목을 채워주세요.

제목 예시: feat : Pull request template 작성

(작성 후 이 안내 문구는 삭제해주세요)

---

작업 내용

• 어떤 기능(또는 수정 사항)을 구현했는지 간략하게 설명해주세요.
• 예) "회원가입 API에 이메일 중복 검사 기능 추가"

---

변경 사항

• 구현한 주요 로직, 클래스, 메서드 등을 bullet 형식으로 기술해주세요.
• 예)
  • UserService.createUser() 메서드 추가
  • @Email 유효성 검증 적용

---

트러블 슈팅

• 구현 중 마주한 문제와 해결 방법을 기술해주세요.
• 예)
  • 문제: @Transactional이 적용되지 않음
  • 해결: 메서드 호출 방식 변경 (this. → AopProxyUtils. 사용)

---

해결해야 할 문제

• 기능은 동작하지만 리팩토링이나 논의가 필요한 부분을 적어주세요.
• 예)D
  • UserController에서 비즈니스 로직 일부 처리 → 서비스로 이전 고려 필요

---

참고 사항

• 기타 공유하고 싶은 정보나 참고한 문서(링크 등)가 있다면 작성해주세요.

---

코드 리뷰 전 확인 체크리스트

• 불필요한 콘솔 로그, 주석 제거
• 커밋 메시지 컨벤션 준수 (type : )
• 기능 정상 동작 확인

Summary by CodeRabbit

• Chores
  • 프로세스·시스템 메트릭 수집을 비활성화하는 구성 토글을 추가했습니다. 운영 환경에서 불필요한 지표를 줄여 대시보드 가독성을 높이고, 에이전트/애플리케이션의 리소스 사용량을 소폭 절감합니다.
  • 구성 파일 포맷을 정리했습니다(섹션 구분 공백, 마지막 줄 개행). 기능적 변경은 없습니다.

[coderabbitai]

Walkthrough

Spring Boot 설정 파일에 프로세스/시스템 메트릭 비활성화 토글 두 개가 추가되었고, 메트릭 섹션 앞에 공백 줄과 파일 말미의 개행이 정리되었습니다. app.redirect 관련 값은 실질적 변경 없이 재기입되었습니다.

Changes

Cohort / File(s)	Change Summary
Configuration (Actuator Metrics) src/main/resources/application.properties	Added management.metrics.enable.process=false, management.metrics.enable.system=false; inserted a blank line before metrics; ensured trailing newline; re-added existing app.redirect.verify.failure without value change.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• fix : 시스템 매트릭 비활성화 #189 — 동일한 영역의 Spring Boot 메트릭 비활성화 토글을 조정하는 변경으로 목적과 범위가 밀접하게 유사함.

Poem

깡충, 깡충—로그를 헤집는 토끼 한 마리,
시계처럼 똑딱, 메트릭 두 개를 잠그고 달리네.
공백 한 줄, 개행 한 번, 질서가 반짝!
리다이렉트 표지판은 제자리에서 눈웃음.
오늘도 설정의 들판에 바람이 산뜻해요. 🐇✨

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title Check	❓ Inconclusive	제목 "Fix/emer"은 너무 일반적이고 모호하여 PR의 핵심 변경사항을 전달하지 못합니다. 이 PR은 application.properties에 management.metrics.enable.process=false 및 management.metrics.enable.system=false를 추가하고 공백과 줄바꿈을 정리하는 구성 변경이므로 현재 제목은 변경의 주요 내용을 반영하지 않습니다. 따라서 제목이 적절한지 결론 내리기 어렵습니다.	제목을 변경해 주세요. 예: "application.properties: 시스템 및 프로세스 메트릭 비활성화" 또는 "config: 시스템·프로세스 메트릭 비활성화". 슬래시나 브랜치명 형태 대신 변경의 핵심을 한 문장으로 요약하면 됩니다.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/emer

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1bcdaa5 and 54a9309.

📒 Files selected for processing (1)

• src/main/resources/application.properties (1 hunks)

🔇 Additional comments (2)

src/main/resources/application.properties (2)

170-171: 확인 요청 — APP_REDIRECT_VERIFY_FAILURE 주입·검증 여부 확인

src/main/resources/application.properties(라인 170)에 app.redirect.verify.failure가 정의되어 있으나 코드베이스 전체 검색(rg) 결과 해당 프로퍼티 참조는 발견되지 않았습니다. 운영 환경에서 APP_REDIRECT_VERIFY_FAILURE가 실제로 주입되는지, 리디렉션 처리 시 대상 도메인에 대한 서버측 화이트리스트/검증 로직이 구현되어 있는지 확인하세요. 현재 리다이렉트 패턴 검색에서는 외부 URL 리다이렉트 호출이 발견되지 않습니다 (ViewController는 내부 '/oauth2/authorization/google'만 사용).

---

172-176: process/system 메트릭 전역 비활성화 — 대시보드/알람 영향 검증 필요, 프로필/환경변수로 제한 적용 권장

검증 결과: build.gradle에서 Spring Boot 3.5.0(플러그인) 및 runtimeOnly 'io.micrometer:micrometer-registry-prometheus'가 확인되어 management.metrics.enable.process/system 키는 유효함. 저장소 내 grafana/alerts/k8s/helm 디렉터리 없음(검색 실패)으로 대시보드·알람 참조 여부는 저장소 외부(Grafana/Alertmanager/운영 룰)에서 확인 필요.

조치 제안:

• 운영/재현 환경에만 비활성화: src/main/resources/application.properties → application-.properties로 옮기거나 환경변수 토글 적용.
• 비활성화 사유(관련 이슈/장애번호)를 주석으로 남김.

파일: src/main/resources/application.properties (lines 172-176)

권장 변경(옵션):

- management.metrics.enable.process=false
- management.metrics.enable.system=false
+ # cgroup 관련 NPE 회피 목적. 환경별로 override 가능.
+ management.metrics.enable.process=${MANAGEMENT_METRICS_ENABLE_PROCESS:false}
+ management.metrics.enable.system=${MANAGEMENT_METRICS_ENABLE_SYSTEM:false}

Tip

👮 Agentic pre-merge checks are now available in preview!

Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.

• Built-in checks – Quickly apply ready-made checks to enforce title conventions, require pull request descriptions that follow templates, validate linked issues for compliance, and more.
• Custom agentic checks – Define your own rules using CodeRabbit’s advanced agentic capabilities to enforce organization-specific policies and workflows. For example, you can instruct CodeRabbit’s agent to verify that API documentation is updated whenever API schema files are modified in a PR. Note: Upto 5 custom checks are currently allowed during the preview period. Pricing for this feature will be announced in a few weeks.

Please see the documentation for more information.

Example:

reviews:
  pre_merge_checks:
    custom_checks:
      - name: "Undocumented Breaking Changes"
        mode: "warning"
        instructions: |
          Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).

Please share your feedback with us on this Discord post.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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