You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Local variables issue: The inspector session is already connected #10278
["Error [ERR_INSPECTOR_ALREADY_CONNECTED]: The inspector session is already connected"," at Session.connect (node:inspector:67:13)","
at cge.configureAndConnect (/node_modules/@sentry/src/integrations/local-variables/local-variables-sync.ts:97:19)","
at Object.setup (/node_modules/@sentry/src/integrations/local-variables/local-variables-sync.ts:344:81)","
at setupIntegration (/node_modules/@sentry/src/integration.ts:119:105)","
at integration (/node_modules/@sentry/src/integration.ts:95:7)","
at Array.forEach (<anonymous>)","
at setupIntegrations (/node_modules/@sentry/src/integration.ts:92:16)","
at _he.setupIntegrations (/node_modules/@sentry/src/baseclient.ts:321:28)","
at oO.bindClient (/node_modules/@sentry/src/hub.ts:171:14)","
at Object.wLt (/node_modules/@sentry/src/sdk.ts:37:7)"]
LocalVariables never really worked for us, or at least I never saw any variables in Sentry interface ever since it was introduced. This session error in particular happened a few versions ago so we disabled it because there were more changes coming and we thought it was a know issue. Then when those changes were released we enabled it again but once again no variables in the interface, so we just ignored waiting for better days. Then updated again and once again the session error at which point we thought it's probably not a known error and that's when we reported.
Activity
Lms24 commentedon Jan 23, 2024
Hi @alfaproject thanks for writing in!
@timfish would you mind taking a look at this when you have time? Thank you!
cbeardsmore commentedon Jan 23, 2024
Seeing the same error after upgrading to 7.95.0 today 😭
timfish commentedon Jan 23, 2024
I'll be looking into this first thing tomorrow. What version did you update from?
cbeardsmore commentedon Jan 24, 2024
Our prod is running the following without seeing this issue:
And saw the first error log today in staging with (not 7.94.1 like I reported earlier sorry):
Sample stack trace if its useful:
alfaproject commentedon Jan 24, 2024
LocalVariables never really worked for us, or at least I never saw any variables in Sentry interface ever since it was introduced. This session error in particular happened a few versions ago so we disabled it because there were more changes coming and we thought it was a know issue. Then when those changes were released we enabled it again but once again no variables in the interface, so we just ignored waiting for better days. Then updated again and once again the session error at which point we thought it's probably not a known error and that's when we reported.
tl;dr never worked in any version
fix(node): LocalVariables integration should use setupOnce (#10307)