mash-up-kr / piikii_Spring

피곤한 모임계획 끝, 키득키득 피키로 ( ͡~ ͜ʖ ͡°)
2 stars 0 forks source link

feat: 예외 로깅 인터셉터 추가 #48

Closed K-Diger closed 1 week ago

K-Diger commented 3 weeks ago

이슈

close #46

변경 사항

스크린샷

부연 설명

위 인터셉터를 적용하면 아래와 같이 에러가 로깅됩니다.

2024-06-18T01:01:16.827+09:00  WARN 38608 --- [piikii] [nio-8080-exec-1] com.piikii.common.logutil.SystemLogger   : {"uri":"/test-throw/illegalArgument","method":"GET","queryString":null,"message":"요청 값이 올바르지 않습니다.","cause":"userId : 4"}

체크리스트

K-Diger commented 3 weeks ago

리베이스 실수로 다날려묵어서 복원한 PR입니다...

K-Diger commented 3 weeks ago

혹시 에러 로그를 슬랙으로 쏘기 위함인 것일까요~? 인터셉터로 구현하지 않아도 보통 Logback appender로 비슷한 기능들을 본 것 같아서 궁금해 질문드립니다~

  • Logback appender 사용하면 기존 Logback log level로 유연하게 관리가 가능할 것 같아서용

오 appender로 hook 알림을 보낼수가있나요??

그렇다면... 이 PR은 닫혀도될 것 같네요,,

KimDoubleB commented 3 weeks ago

앗 ㅋㅋㅋ 정확하진 않을 수 있습니다. 보통 appender 구현체 만들어서 구현이 가능했던 것 같아서... logback appender slack라고 검색하니깐 여러가지 나오는 듯?ㅋㅋㅋ 원래는 appender 기능으로 log storage로 쏜다거나 했던 것 같은데 일단 slack 붙여놔도 좋을 듯ㅋㅋ