EMFILE: too many open files, uv_resident_set_memory in _updateContext

[ghost]

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/nextjs

SDK Version

7.77.0

Framework Version

Next 14.0.1

Link to Sentry event

https://sentry.io/organizations/thinkaidium/issues/4598335869/?project=4504118550528000&referrer=github_integration

SDK Setup

client:

init({
    dsn: SENTRY_DSN,
    environment: process.env['ENVIRONMENT'],
    tracesSampleRate: 0.1,

    // This sets the sample rate to be 10%. You may want this to be 100% while
    // in development and sample at a lower rate in production
    replaysSessionSampleRate: 0.1,
    // If the entire session is not sampled, use the below sample rate to sample
    // sessions when an error occurs.
    replaysOnErrorSampleRate: 1.0,
    integrations: [new Replay()],
  });

edge:

init({
    dsn: SENTRY_DSN,
    environment: process.env['ENVIRONMENT'],
    tracesSampleRate: 0.1,
  });

server:

init({
    dsn: SENTRY_DSN,
    environment: process.env['ENVIRONMENT'],
    tracesSampleRate: 0.1,
  });

Steps to Reproduce

I'm not sure. You can get the call stack from the sentry link attached.

node:internal/process/per_thread in process.memoryUsage at line 172:5
/var/task/node_modules/@sentry/node/cjs/integrations/context.js in Context._updateContext at line 76:41
/var/task/node_modules/@sentry/node/cjs/integrations/context.js in Context.addContext at line 56:33

After a bit of a research from my side, I see that the community uses 'graceful-fs'/'node-fs-extra' instead of 'fs' to prevent EM File errors.

Expected Result

No errors?

Actual Result

Throws this error.

[lforst]

Hm, the error was thrown on this line:

sentry-javascript/packages/node/src/integrations/context.ts

Line 96 in 9762348

contexts.app.app_memory = process.memoryUsage().rss;

I don't know if this is fixable from within the SDK. To me it looks like either you or the hosting provider had too many FDs open on the system.

[ghost]

From some online research, I saw that 'fs' doesn't take care of "EM file errors", which is seen here. I see that you're using 'fs' in multiple places to deal with files. Have you considered changing it to 'graceful-fs' or 'node-fs-extra'?

[lforst]

@misuvii we generally steer away from dependencies, especially in low-level-ish SDKs like @sentry/node. The thing is, in the line we threw in, we didn't even use fs. It's a node internal API that threw this error. I wouldn't know how to fix this except for try catching here, however that feels weird considering this is just node internal API that should generally not throw in the first place.

[AbhiPrasad]

Closing this PR as cleanup, we've made many adjustments in the latest version of the SDK. If the issue is still happening after upgrading the SDK to the latest version, please open a new issue!