[Pino integration] - `pino-http` logs for sentry not working

[jalbertsr]

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 - express

SDK Version

@sentry/node: "^10.19.0"

Framework Version

express@4.21.2 pino-http@9.0.0 node@20.17.0

Link to Sentry event

https://swipoo.sentry.io/explore/logs/?environment=production&logsFields=timestamp&logsFields=message&logsSortBys=-timestamp&project=4504836595777537&statsPeriod=7d

Reproduction Example/SDK Setup

instrument.js

import * as Sentry from '@sentry/node';

import { envVariables } from './src/config/envVariables';

Sentry.init({
  dsn: envVariables.SENTRY_DSN,
  enableLogs: true,
  integrations: [Sentry.pinoIntegration()],
  tracesSampleRate: 1,
  environment: envVariables.ENV === 'PROD' ? 'production' : 'development',
});

index.js

import './instrument';
import pino from 'pino-http';
import * as Sentry from '@sentry/node';

const app = express();
app.use(pino());

Steps to Reproduce

1. Install express CRUD API with pino-http as logger.
2. Install latest Sentry SDK
3. No pino logs :(

Expected Result

I expect all logs from pino to show.

Example:

$ node example.js | pino-pretty
[2016-03-31T16:53:21.079Z] INFO (46316 on MBP-di-Matteo): something else
    req: {
      "id": 1,
      "method": "GET",
      "url": "/",
      "headers": {
        "host": "localhost:3000",
        "user-agent": "curl/7.43.0",
        "accept": "*/*"
      },
      "remoteAddress": "::1",
      "remotePort": 64386
    }
[2016-03-31T16:53:21.087Z] INFO (46316 on MBP-di-Matteo): request completed
    res: {
      "statusCode": 200,
      "header": "HTTP/1.1 200 OK\r\nX-Powered-By: restify\r\nContent-Type: text/html; charset=utf-8\r\nContent-Length: 11\r\nETag: W/\"b-XrY7u+Ae7tCTyyK7j1rNww\"\r\nDate: Thu, 31 Mar 2016 16:53:21 GMT\r\nConnection: keep-alive\r\n\r\n"
    }
    responseTime: 10
    req: {
      "id": 1,
      "method": "GET",
      "url": "/",
      "headers": {
        "host": "localhost:3000",
        "user-agent": "curl/7.43.0",
        "accept": "*/*"
      },
      "remoteAddress": "::1",
      "remotePort": 64386
    }

Actual Result

Only logs done via Sentry.logger.info("Hello world") show,

Additional Context

No response

[linear]

JS-1060

[JPeer264]

Hey thanks for your issue. I just set up a basic Express app with CommonJS and the logs are showing up as expected. Do you run your express server via CJS or ESM? Also to confirm do any traces for your express server show up on your Sentry environment?

[jalbertsr]

Hey @JPeer264 thanks for your help, the app is currently an ESM application that uses Babel for transpilation to support ESM import/export statements in Node.js (a bit of legacy setup).

We run it using babel-node index.js We’re aware this isn't ideal for production, it's an older app, and we've added a migration to native ESM (without Babel) to our roadmap.

currently we have:

"@babel/core": "^7.28.4",
"@babel/node": "^7.28.0",
"@babel/preset-env": "^7.28.3",

[JPeer264]

I am not super familiar with babel-node, but since you are running ESM it needs to be loaded with the --import flag: node --import ./instrument.js index.js. I couldn't find an --import flag on their docs though.

You could try to use it like following and see if it works (remember you don't need to import the instrumentation in the index.js then anymore):

NODE_OPTIONS="--import ./instrument.js" babel-node index.js

Could you confirm if that works?