feat : pino child 추가 #224
Merged
feat : pino child 추가 #224
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
soomtong
requested review from
9w and
TaehuiKim
and removed request for
soomtong
|
child 로거를 통해서 추가로그를 봐야하는게 어떤 이점이 있어요? 뭔가 정형화된 audit 로그를 구성하려는 거면 다른 패키지나 다른 구성으로 준비하는게 어떨까 싶어요. |
기존에는 obj 데이터를 로깅하려 할때 msg에 문자열 형태로 변환하여 보여지고 있어 예를 들면 |
|
아하, 매우 유용하겠네요! |
moonimooni
reviewed
Aug 7, 2024
src/logger/logger.interface.ts
Outdated
| @@ -8,4 +9,5 @@ export interface Logger { | |||
| warn(msgTemplate?: string, ...args: unknown[]): void; | |||
| error(msgTemplate?: string, ...args: unknown[]): void; | |||
| log(msgTemplate?: string, ...args: unknown[]): void; | |||
| child(options: Record<string, unknown>): pino.Logger; | |||
There was a problem hiding this comment.
피노 디펜던시는 구현체에만 존재하고 페블즈 인터페이스는 독립적으로 존재하는 게 본래 의도에 맞습니다.
급한 건이라면 추후 개선해도 괜찮겠지만 아니라면 수정되었으면 해요.
구현체에서 constructor 오버로딩을 통해 child 메소드에서 Logger 타입을 리턴할 수 있을 거 같아 보여요.
테스트도 붙여주시면 좋습니다. (부모 로거와 차일드 로거의 객체가 구분되는가 같은거..)
There was a problem hiding this comment.
overloading 해서 사용자가 체이닝 해서 사용하는 방법과
기존 사용성을 유지하는 flat 을 추가 하였습니다.
의견 부탁드려요 🙏
jo-seongjun
approved these changes
Aug 7, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR 의 종류는 어떤 것인가요?
무엇을 어떻게 변경했나요?
log 가독성을 높이기 위해 pino.child 추가
코드 변경을 이해하기 위한 배경지식이 필요하다면 설명 해주세요.
https://github.com/pinojs/pino
디펜던시 변경이 있나요?
없음
어떻게 테스트 하셨나요?
스모크, 스웨거
코드의 실행결과를 볼 수 있는 로그나 이미지가 있다면 첨부해주세요.