[Feat/#26] Log 출력 기능 구현 - [merged]

[TaeYeongKwak]

Merges feat/#26 -> dev

📋 구현 내용

요청에 대한 Log 출력 기능 구현

• Controller Log 출력 구현
• Error Log 출력 구현
• Message Broker Log 출력 구현

🔍 테스트 케이스

• 성공
• 오류 발생

📢 코멘트

• 일단 Api Log는 Controller에서 처리하기로 하여 controller package에 있는 모든 클래스에 적용하도록 pointcut을 작성하였습니다.
• Message Broker Producer나 Consumer에 로그를 출력하려면 해당 메서드에 MessageLog 어노테이션을 추가하고 각각의 exchange와 queue의 이름을 인자로 넣어주시면 됩니다.

@tommysgit @AnTaeWook @muyongKim

[TaeYeongKwak]

added 32 commits

• 894ae2e0...169461ca - 31 commits from branch dev
• 74add59c - Conflicts: build.gradle

Compare with previous version

[TaeYeongKwak]

In GitLab by @AnTaeWook on Feb 16, 2023, 13:41

LGTM :thumbsup:

[TaeYeongKwak]

In GitLab by @AnTaeWook on Feb 16, 2023, 13:41

approved this merge request

[TaeYeongKwak]

In GitLab by @tommysgit on Feb 16, 2023, 13:43

Commented on src/main/java/com/gabia/weat/gcellapiserver/dto/log/LogFormatFactory.java line 17

해당 변수를 정적 변수로 선언한 이유가 무엇인가요?

[TaeYeongKwak]

해당 변수는 traceId의 길이를 설정하기 위한 값인데 후에 길이가 변경될 수 도 있을 것 같아 관리하기 편하도록 일단 정적 변수로 선언하였습니다.

[TaeYeongKwak]

In GitLab by @tommysgit on Feb 16, 2023, 13:55

Commented on src/main/java/com/gabia/weat/gcellapiserver/dto/log/LogFormatFactory.java line 17

클래스 안에서만 사용된다면 빼는게 어떤가 싶어서 얘기하였습니다.

[TaeYeongKwak]

changed this line in version 3 of the diff

[TaeYeongKwak]

added 2 commits

• fabc6310 - Refactor : TRACE_ID_LENGTH 변수 static 제거
• f381d8c7 - Merge remote-tracking branch 'origin/feat/#26' into feat/#26

Compare with previous version

[TaeYeongKwak]

해당 부분 변경 완료하였습니다.

[TaeYeongKwak]

resolved all threads

[TaeYeongKwak]

In GitLab by @tommysgit on Feb 16, 2023, 14:02

resolved all threads

[TaeYeongKwak]

In GitLab by @tommysgit on Feb 16, 2023, 14:08

approved this merge request

[TaeYeongKwak]

mentioned in commit 1e7928292ef4b669cbfcb3e9837babb28e9b5c99