fix: 4xx 비즈니스 예외를 WARN 레벨로 기록하도록 수정

[pooreumjung]

#️⃣ Issue Number

• resolved fix: ControllerLogAspect가 4xx 비즈니스 예외를 ERROR 레벨로 기록하는 문제 #485

📝 요약(Summary)

• 문제: ControllerLogAspect가 컨트롤러에서 발생한 모든 예외를 log.error(..., e)로 기록했습니다. 그래서 NOT_FOUND_MENU 같은 정상적인 4xx 비즈니스 예외도 ERROR 레벨에 스택트레이스까지 남았습니다.
  • ERROR 로그가 실제 서버 장애보다 부풀려집니다. 나중에 Loki 기반 ERROR 알림(feature: 로그/모니터링 보류 개선 항목 정리 #484)을 걸면 4xx 때문에 알림이 울리는 노이즈가 생깁니다.
  • 스택트레이스 때문에 로그 용량도 불필요하게 늘어납니다.
• 변경: 이미 Slack 알림에 쓰고 있던 조건(5xx 여부)으로 로그 레벨도 나눴습니다.
  • 5xx BaseException과 예상하지 못한 예외: ERROR에 스택트레이스를 남기고 Slack으로 알립니다. 기존과 같습니다.
  • 4xx BaseException: WARN 한 줄로 남깁니다. 스택트레이스는 남기지 않고, type과 cause 메시지는 그대로 유지합니다.
  • 조건 메서드 이름을 shouldNotifySlack에서 isServerError로 바꿨습니다. 이제 로그 레벨과 Slack 알림 두 곳에서 같이 쓰기 때문입니다.

💬 공유사항 to 리뷰어

• 5xx 동작은 바뀌지 않았습니다. ERROR 로그와 스택트레이스, Slack 알림 조건이 기존과 같습니다.
• "BaseException이 아니면 5xx"로 판단합니다. 컨트롤러에서 @PreAuthorize나 ResponseStatusException처럼 BaseException이 아니면서 4xx로 바뀌는 예외를 던지는 코드가 없는 것을 확인했습니다. 앞으로 이런 예외를 도입하면 이 조건도 함께 손봐야 합니다.
• 4xx에서도 Aspect 로그는 유지합니다. #479에서 켠 Resolved [...] WARN 로그는 BaseException에 메시지가 없어서 클래스 이름만 찍힙니다. 그래서 어떤 에러였는지(cause=...)는 Aspect 로그에만 남습니다. 4xx 요청 하나에 WARN 두 줄이 남는 셈이고, 둘은 같은 requestId로 묶입니다.
• 테스트:
  • Logback ListAppender로 실제 찍힌 로그를 잡아서 검증하는 테스트 2개를 추가했습니다. 4xx는 WARN이고 스택트레이스가 없는지, 5xx는 ERROR이고 스택트레이스가 있는지 확인합니다.
  • 전체 638개가 통과했습니다.
• dev 배포 후 확인할 것: 4xx가 나는 요청을 보낸 뒤 {service_name="eatssu-dev"} | json | level="ERROR"로 조회해서 해당 요청이 나오지 않는지, level="WARN"으로는 나오는지 확인합니다.

✅ PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

• 커밋 메시지 컨벤션에 맞게 작성했습니다.
• 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

🤖 Generated with Claude Code

[github-actions]

📊 테스트 커버리지 리포트

Overall Project	95.62%	🍏
Files changed	100%	🍏

File	Coverage
ControllerLogAspect.java	98.49%	🍏