Closed
Description
Describe request or inquiry
- Both JavaScript/Node.js microservices (
paymentservice
andcurrencyservice
) useconsole.log
. - See files:
- Let's use
pino
instead. - We're already using
pino
in other JavaScript files. - This idea originated from 665.
What purpose/environment will this feature serve?
- See pull-request 66 — where the use of
pino
was introduced — for reasoning. - Basically, we want our logs to be formatted as JSON — for Stackdriver. See JSON Structured logs for Stackdriver Logging #47.
Activity
NimJay commentedon Jul 14, 2022
Update (2022 July)
Just adding an update to cool this issue down (i.e., remove it from our team's out-of-SLO list).
No progress updates and this issue is still very low priority.
I wonder if this ticket will be addressed (indirectly) by #422 (i.e., will switching to OpenTelemetry render this issue irrelevant?).
aarushisoni commentedon Dec 7, 2022
Hi I want to contribute to this issue please assign me this issue
NimJay commentedon Dec 7, 2022
Hi @aarushisoni,
Thanks again for your interest.
This is definitely a good first issue to work on. :)
I've assigned it to you.
TonyBhargav commentedon Jan 22, 2023
Hi @NimJay since no progress in this issue can I get assigned to this issue. I'll be happy to work on this.
@aarushisoni are you still working on this?
andresmc98 commentedon Apr 25, 2023
Hi @NimJay
is someone working on this issue? i would like give it a shot c:
13 remaining items