Use pino instead of console.log

[NimJay]

Describe request or inquiry

• Both JavaScript/Node.js microservices (paymentservice and currencyservice) use console.log.
• See files:
  • /src/paymentservice/index.js
  • /src/currencyservice/server.js
• 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.

[NimJay]

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]

Hi I want to contribute to this issue please assign me this issue

[NimJay]

Hi @aarushisoni,
Thanks again for your interest.
This is definitely a good first issue to work on. :)
I've assigned it to you.

[TonyBhargav]

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]

Hi @NimJay
is someone working on this issue? i would like give it a shot c:

[NimJay]

Hi @andresmc98, :)

I don't think anyone was working on this. Correct my if I'm wrong, @TonyBhargav. :)
I just assigned it to you (@andresmc98).

Just a heads-up: please see before starting any work:

• CONTRIBUTING.md
• development-guide.md

Thank you all, for your interest in contributing.

[andresmc98]

Thank you,
so i'll checks the docs first

[DracFiendMG]

Hi @NimJay, can you assign this one to me? I've already made relevant changes to both index.js and server.js files on my local. I want to confirm the changes with you.

[DracFiendMG]

@NimJay, I just noticed that /src/currencyservice/server.js already has pino implemented, can you confirm?

[DracFiendMG]

@NimJay, I've raised a PR: #2262
Can you look into it and let me know if I'm missing something?

[DracFiendMG]

@NimJay, I've already committed my changes can you look into it