MySQL tracing integration breaks non-callback queries

[chriswiggins]

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

7.73.0

Framework Version

node 18

Link to Sentry event

No response

SDK Setup

sentry.init({
  dsn: 'https://111111@sentry.io/11111',
  debug: true,
  enabled: true,
  environment: '123',
  release: '111',
  serverName: '111',
  tracesSampleRate: 0
});

Steps to Reproduce

We use the node mysql library with streams. This doesn't require a callback being passed to the query function.

We just updated the packages of one of our services, which has bought in an update to sentry and the tracing modules, but this completely broke our deployment.

The last version we were on was 7.40.0 where the following code snippet works. This doesn't work on the latest release.

const sentry = require('@sentry/node');
require('@sentry/tracing');

const { createPool } = require('mysql');

sentry.init({
  dsn: 'https://111111@sentry.io/11111',
  debug: true,
  enabled: true,
  environment: '123',
  release: '111',
  serverName: '111',
  tracesSampleRate: 0
});


const mysql = createPool({
  connectionLimit: 10,
  database: 'user',
  host: 'localhost',
  password: 'user',
  user: 'user'
});

const query = mysql.query('SELECT * from user');
query.on('result', console.log);  

You can get the above code snippet to produce rows from the query by setting enabled: false or not requiring the @sentry/tracing plugin

Expected Result

See above

Actual Result

See above

[mydea]

Hmm, I see, maybe this is related to #8923. Could you try updating to 7.66.0 and see if it still works there? Then we can fully pin this to that PR.

[mydea]

I opened a PR to hopefully fix this issue here: #9222

[mydea]

This should be fixed in 7.74.0 - can you give it a try? If the problem remains, please reopen this issue - thank you!