Configure Pino.js to work with pm2

Describe the bug

Pino.js has an issue with pm2 if you manually set the destination to descriptor 1. Because pm2 overwrites process.stdout.write with its own function. see https://github.com/pinojs/pino/issues/491#issuecomment-414075492

Pino fixed this issue in https://github.com/pinojs/pino/pull/493/files if you don’t specify the destination when writing to stdout.

https://github.com/pinojs/pino/issues/491

To Reproduce

See this comment, https://github.com/pinojs/pino/issues/491#issuecomment-414043388

Expected behavior

Let pino.js handle the magic by removing the destination.

  const logger = pino(pinoConfig, destination);
=>
  const logger = pino(pinoConfig);

Configuration File (cat ~/.config/verdaccio/config.yaml)

logs: { type: stdout, format: pretty, level: http }

Environment information

verdaccio: 5.x

[amitgilad3]

Thanks @favoyang for the info, i already have a fix in place.

[stale[bot]]

Hi pal 👋🏼! This issue has gone quiet 😶.

We get a lot of issues, so we currently close issues after 180 days of inactivity. It’s been at least 90c days since the last update here. If we missed this issue or if you want to keep it open, please reply here. You can also add/suggest the label “discuss” to keep this issue open! As a friendly reminder: the best way to see this issue, or any other, fixed is to open a Pull Request. Check out https://github.com/verdaccio/contributing for more information about opening PRs, triaging issues, and contributing!

Thanks for being a part of the Verdaccio community! 💘