Memory leak in the default RedisIntegration

[EDjur]

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

8.36.0

Framework Version

No response

Link to Sentry event

No response

Reproduction Example/SDK Setup

import * as Sentry from '@sentry/node';

...

Sentry.init({
            dsn,
            environment,
            release,
            tracesSampleRate: 1.0,
            maxValueLength: 400,
            profilesSampleRate
});

Steps to Reproduce

Basically we just set up Sentry with our Fastify server that also has a redis component. It seems that the RedisIntegration (which is enabled by default), causes a memory leak somewhere.

I don't know enough about Sentry internals to dig much deeper I'm afraid. All I know is the memory leak stops when I disable the RedisIntegration.

Do let me know if there's any additional details I can try dig up for you.

Expected Result

No memory leaks.

Actual Result

The graph summarizes the problem pretty well. This is pod memory usage.

Orange: Without RedisIntegration
Blue: With RedisIntegration

[AbhiPrasad]

Hey @EDjur thanks for writing in! We'll get right on this, memory issues are very high priority for us. Sorry for the trouble.

Could you share the version of redis you are using? As well as if you are using ioredis or redis?

@timfish mind taking a look when you get some time (given you're memory profiling already for the local variables issue 😄)

[EDjur]

Hey, and thanks for the swift reply 👋

I'm using redis @ 4.7.0.

And to be clear, this isn't blocking anything for me personally since I don't need Sentry for my redis component. But it's possible I'm not the only one seeing this problem of course :)

And again, let me know if there's any other info needed. I imagine it's not as simple as the RedisIntegration leaking by itself in isolation.

[lforst]

@EDjur it would be amazing if you could provide a memory snapshot so we can see what references are being held that consume the memory.