Pino integration - Pino versions below 9.10 doesn't show up on Sentry logs

[mrljsh]

Is there an existing issue for this?

• I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
• I have reviewed the documentation https://docs.sentry.io/
• I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases

How do you use Sentry?

Sentry Saas (sentry.io)

Which SDK are you using?

@sentry/node - fastify

SDK Version

10.19.0

Framework Version

Fastify v4.29.0

Link to Sentry event

No response

Reproduction Example/SDK Setup

No response

Steps to Reproduce

1. Install pino@9.7.0
2. Implement Sentry pino integration
3. Start the server, try to log things, they won't show up on Sentry logs

Expected Result

As per documentation, any version above 9.0.0 -> Fastify server logs should appear on Sentry dashboard

Actual Result

Logs wont show unless pino is updated to ^9.10.0 version. Sentry.log methods work as usual.

Additional Context

• I am not sure what's the reason behind this, but I think you should set info in docs that recommended version is ^9.10.0.
• Also, maybe add some logging to the integration, when we enable debug: true in Sentry.init, we should have some feedback if logs are successfully sent to the Sentry or not.

[linear]

JS-1048

[logaretm]

The v9.10 release introduced asJSON tracing channels which are used by the Pino integration.

The injected channel should work tho for < 9.10. I will check that out and update the docs if necessary.

[logaretm]

@mrljsh Can you please provide a minimal reproduction?

I tried creating a new node project with pino@9.7 and it works well as expected with the injected channels.

[mrljsh]

Sentry.init({
    dsn: opts.dsn,
    environment: config.cloudEnv,

    // Add Tracing by setting tracesSampleRate
    tracesSampleRate: 1.0,

    // Set sampling rate for profiling
    // This is relative to tracesSampleRate
    profilesSampleRate: 1.0,

    // Add request headers and IP to the Sentry event
    sendDefaultPii: true,

    // Logs
    enableLogs: true,

    // Add integrations
    integrations: [
      nodeProfilingIntegration(),
      Sentry.knexIntegration(),
      Sentry.pinoIntegration(),
    ],
  });

And app.js

const server = Fastify({
    logger: true,
  });

  Sentry.setupFastifyErrorHandler(server);

I don't know why, but I had this problem for a few days and I saw one of the comments about new asJSON channels. So I ran `pnpm update pino@latest" and it started working without any issues.