feat: MongoDB Tracing Support

[kamilogorek]

MongoDB tracing support.

Usage:

constSentry=require("@sentry/node");constTracing=require("@sentry/tracing");constmongodb=require("mongodb");Sentry.init({dsn: "__PUBLIC_DSN__",integrations: [newTracing.Integrations.Mongo(),],tracesSampleRate: 1.0,});constclient=newmongodb.MongoClient("mongodb://mongo:docker@localhost:27017");// the rest of the code, for raw-node code we need manual instrumentation, eg.asyncfunctionrun(){consttransaction=Sentry.startTransaction({op: "transaction",name: "My Transaction",});Sentry.configureScope((scope)=>{scope.setSpan(transaction);});try{awaitclient.connect();constdatabase=awaitclient.db("admin");constcollection=database.collection("movies");awaitcollection.insertOne({title: "Rick and Morty"});awaitcollection.findOne({title: "Back to the Future"});awaitcollection.updateOne({title: "Back to the Future"},{$set: {title: "South Park"}});awaitcollection.findOne({title: "South Park"});}finally{if(transaction)transaction.finish();awaitclient.close();}}run().catch(console.dir);

Works nicely with Express too:

constSentry=require("@sentry/node");constTracing=require("@sentry/tracing");constmongodb=require("mongodb");constexpress=require("express");Sentry.init({dsn: "__PUBLIC_DSN__",integrations: [newSentry.Integrations.Http({tracing: true}),newTracing.Integrations.Express({app: express.Router,methods: ["get"],}),newTracing.Integrations.Mongo(),],tracesSampleRate: 1.0,});constapp=express();constclient=newmongodb.MongoClient("mongodb://mongo:docker@localhost:27017");client.connect();app.use(Sentry.Handlers.requestHandler());app.use(Sentry.Handlers.tracingHandler());app.use(express.json());app.get("/hi",asyncfunctionsayHi(req,res,next){try{constdatabase=awaitclient.db("admin");constcollection=database.collection("movies");awaitcollection.insertOne({title: "Rick and Morty"});constresult=awaitcollection.findOne({title: "Rick and Morty"});res.send(result);}catch(err){next(err);}});app.use(Sentry.Handlers.errorHandler());app.listen(3000);

[github-actions]

size-limit report

Path	Size
@sentry/browser - CDN Bundle (gzipped)	19.73 KB (-0.01% 🔽)
@sentry/browser - Webpack	20.6 KB (0%)
@sentry/react - Webpack	20.6 KB (0%)
@sentry/browser + @sentry/tracing - CDN Bundle (gzipped)	26.88 KB (0%)

[lobsterkatie]

Why pass in getCurrentHub?

[HazAT]

This is how integrations work internally, the function will be injected from outside and we should use that vs. global getCurrentHub.

[lobsterkatie]

Is "describe" your word or theirs? If it's yours, I wonder if we might name this option differently, to a) differentiate it from the description field in a transaction, and b) indicate that what it's really about is recording the operation's inputs.

[HazAT]

I think describe is used in database lingo.

[HazAT]

Merging this now, has also few other fixes.

[Copdate-Copdate]

Hi, question about MongoDB support.

I'm currently using Mongoose inside an Express app and when adding new Tracing.Integrations.Mongo() to integrations array in Sentry.init, I got the following error:

UnhandledPromiseRejectionWarning: TypeError: Tracing.Integrations.Mongo is not a constructor

Do I have to pass some config or it just doesn't support Mongoose right now?

[sol-austin]

I can't seem to get this working with Mongoose. I've added Tracing.Integrations.Mongo() to the integrations array without receiving an error message but it doesn't seem to be tracking database requests at all.

Has anyone successfully got this working with Mongoose yet?

[KevinEdry]

Same here, would love to see mongoose support for this.