opentelemetry-demo icon indicating copy to clipboard operation
opentelemetry-demo copied to clipboard

paymentsservice: structure logs

Open naseemkullah opened this issue 1 year ago • 1 comments

Use Pino's mergingObject parameter to add context as JSON rather than in the message parameter, which is now more generic (like an event name).

Also factor out logger so it is instantiated once for the whole service and upgrade pino.

Also remove unneeded comments.

naseemkullah avatar Oct 22 '22 23:10 naseemkullah

please remove the hardcoded default port, the rest looks great.

Thanks!

done, my pleasure

naseemkullah avatar Oct 23 '22 13:10 naseemkullah

LGTM!

mic-max avatar Oct 28 '22 18:10 mic-max