Express Integration is not exported

[Igor-Lopes]

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

SDK Version

5.3.0

Framework Version

express 4.15.13

Link to Sentry event

No response

SDK Setup

  Sentry.init({
    dsn: 'https://',
    integrations: [
      new Sentry.Integrations.Http({ tracing: true }),
      new Sentry.Integrations.Express({ app }),
      nodeProfilingIntegration()
    ],

    // Performance Monitoring
    tracesSampleRate: 1.0, //  Capture 100% of the transactions
    // Set sampling rate for profiling - this is relative to tracesSampleRate
    profilesSampleRate: 1.0
  })

  app.use(Sentry.Handlers.requestHandler())
  app.use(Sentry.Handlers.tracingHandler())
  app.use(Sentry.Handlers.errorHandler()) 

Steps to Reproduce

1. Try to setup Sentry for Express.js just like instructed from Sentry Dashboard
2. You'll get this error: TypeError: Sentry.Integrations.Express is not a constructor
3. If you remove this line new Sentry.Integrations.Express({ app }), app will not break.

Expected Result

Sentry Express Integration should work.

Actual Result

TypeError: Sentry.Integrations.Express is not a constructor.

It seems like Express Integration is not exported. You can easily check this by calling console.log(Sentry). Express is not available under Integrations objects. That's why new Sentry.Integrations.Http({ tracing: true }) will work, but new Sentry.Integrations.Express({ app }) won't. If you see the result of this log, Http integration is exported, thus, working.

[mydea]

Hello,

you are using SDK version 5, which is quite old. All our docs etc. are for the current version of the SDK (v7 or very soon v8). Please update your SDK. You can find help on migrating to newer SDK versions here: https://docs.sentry.io/platforms/javascript/guides/node/migration/

[Igor-Lopes]

Sorry I've thought it was installed the latest version. However, if you run yarn such as instructed on sentry website, it will install version 5 by default. You have to specify the latest version to force install it.

[Igor-Lopes]

@mydea I think Sentry Dashboard should be updated to follow this guide:
https://docs.sentry.io/platforms/javascript/guides/express/migration/v7-to-v8/#important-changes

Currently on Sentry Dashboard:

Sentry.init({
  dsn: "https://",
  integrations: [
    // enable HTTP calls tracing
    new Sentry.Integrations.Http({ tracing: true }),
    // enable Express.js middleware tracing
    new Sentry.Integrations.Express({ app }),
    nodeProfilingIntegration(),
  ],
  // Performance Monitoring
  tracesSampleRate: 1.0, //  Capture 100% of the transactions
  // Set sampling rate for profiling - this is relative to tracesSampleRate
  profilesSampleRate: 1.0,
});

[E2-Meghanadhan]

This isssue still persists..in v8.. I think the syntax must be different for integrations..
Here the mentioned. but not perfect example

[mydea]

What exactly are you referring to with "Sentry Dashboard"? The v7 format should not be recommended anywhere in docs anymore, maybe we missed something somewhere. In v8, you do not need to add any integrations, all of this is auto-added for you. All you need to do for express is documented here: https://docs.sentry.io/platforms/javascript/guides/express/

// instrument.js
import * as Sentry from "@sentry/node";
import { nodeProfilingIntegration } from '@sentry/profiling-node';

// Ensure to call this before importing any other modules!
Sentry.init({
  dsn: "xxx",
  tracesSampleRate: 1.0,
});

// app.js
const app = express();

// Add your routes, etc.

// Add this after all routes,
// but before any and other error-handling middlewares are defined
Sentry.setupExpressErrorHandler(app);

[E2-Meghanadhan]

Yeah, It's working fine.. Thanks

…

On Mon, 30 Dec, 2024, 4:19 pm Francesco Gringl-Novy, < ***@***.***> wrote: What exactly are you referring to with "Sentry Dashboard"? The v7 format should not be recommended anywhere in docs anymore, maybe we missed something somewhere. In v8, you do not need to add any integrations, all of this is auto-added for you. All you need to do for express is documented here: https://docs.sentry.io/platforms/javascript/guides/express/ // instrument.jsimport * as Sentry from ***@***.***/node";import { nodeProfilingIntegration } from ***@***.***/profiling-node'; // Ensure to call this before importing any other modules!Sentry.init({ dsn: "xxx", tracesSampleRate: 1.0,}); // app.jsconst app = express(); // Add your routes, etc. // Add this after all routes,// but before any and other error-handling middlewares are definedSentry.setupExpressErrorHandler(app); — Reply to this email directly, view it on GitHub <#11925 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/A4JZXYYTL4VLHBA3V5ARDAT2IEQLXAVCNFSM6AAAAABUEZFQKSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNRVGMYTMMRYGY> . You are receiving this because you commented.Message ID: ***@***.***>