로그 설정

[JJ503]

• closed [Feat] 로그 설정 #75

dev 서버에서의 로깅 설정 진행했습니다.
이전 회의에서 이야기한 바와 같이 warn과 error에 대해 아래와 같은 형식으로 진행했습니다. (에러는 아래 내용에 stacktrace가 추가됩니다. 길어서 생략)

{
    "timestamp":"2024-02-26 14:00:24.169",
    "method":"POST",
    "version":"1",
    "uri":"/auth/login/oauth/kakao",
    "message":"InvalidAuthorizationTokenException : 유효하지 않은 토큰입니다."
}

바뀐 점이라면 version이 추가되고 사용자 아이디가 없어졌습니다.
이 부분에 대해 의견을 여쭤보고 싶었습니다.
일단, 버전의 경우 저희가 나중에 여러 버전이 생기게 되면 어떤 버전에서 발생한 문제인지 알아야 문제의 원인을 파악하기에 더 좋을 것 같아 추가했습니다.
그리고 사용자 아이디의 경우 어떻게 가져올 수 있을지에 대해 좀 더 고민해봐야 할 것 같습니다. 당장은 어떤 식으로 가져오는 것이 효율적일지 잘 모르겠기도 하고, 사용자 아이디를 사용하지 않는 부분이라면 어떻게 해야 할지에 대한 고민들이 있는 상태라 이번 pr에서는 제외하려고 합니다. 괜찮을까요?

추가로 기존 저희의 컨벤션은 private 메서드들에 대해 여러 곳에서 사용하더라도 첫 번째로 사용한 메서드 바로 아래 위치하기로 했습니다.
그런데 GlobalExceptionHandler에서 해당 컨벤션을 사용하면 오히려 가독성이 좋지 않을 것이라 판단해 모두 아래에 위치해두었는데 괜찮을까요?

[sonarqubecloud]

Quality Gate passed

Issues
28 New issues

Measures
0 Security Hotspots
97.1% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

[jhsseonn]

로그 설정 고생하셨습니다!
딱히 수정할 부분이 없어서 바로 approve 하겠습니다!
로그에서 버전 정보도 �볼 수 있도록 한 부분 좋은 것 같습니다.
아래는 질문 주신 부분에 대한 답변입니다.

1. 사용자 아이디 가져오기
   확실히 사용자 아이디를 가져오기에는 사용자 아이디를 받아오지 않는 메서드에서는 처리하기 어려울 수 있겠군요.. api 호출시에 사용자 정보를 가져올 수 있도록 하든가 아니면 사용자 아이디 외에도 호출한 사용자를 식별할 수 있는 정보(디바이스 토큰 정보 등)를 가져올 수 있는 좀 더 효율적인 방법을 고려해봐야할 것 같습니다. 추후 논의를 좀 더 거친 후에 반영하면 좋을 것 같습니다!
2. GlobalExceptionHandler에서의 메서드 정렬
   저도 로그 설정 관련한 메서든는 GlobalExceptionHandler에서 맨 아래에 빼는 것에 동의합니다. 아무래도 기능 관련한 프로덕션 코드라기 보다는 테스트에 가깝다는 생각이 들어서 아예 빼는 것이 좋을 것 같다고 생각했습니다.