From ac1c48b71036587a6b5698edacc31c692d9fdc32 Mon Sep 17 00:00:00 2001 From: Luca Forstner Date: Wed, 24 Apr 2024 08:11:31 +0000 Subject: [PATCH] feat(core): Increase default transport buffer size from 30 to 64 Nowadays, the SDK simply sends more data and we are seeing overflows in our E2E tests which do not produce an unreasonable amount of events. Dropped events can lead to broken traces and confusing data inside Sentry, which is why I think bumping this is reasonable. We make this under the assumption that an event in the transport holds at max something like 5MB of memory, which would bump the memory consumption from 150MB to 320MB. --- packages/core/src/transports/base.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/core/src/transports/base.ts b/packages/core/src/transports/base.ts index 0ad22d3d7a15..965125590896 100644 --- a/packages/core/src/transports/base.ts +++ b/packages/core/src/transports/base.ts @@ -26,7 +26,7 @@ import { import { DEBUG_BUILD } from '../debug-build'; -export const DEFAULT_TRANSPORT_BUFFER_SIZE = 30; +export const DEFAULT_TRANSPORT_BUFFER_SIZE = 64; /** * Creates an instance of a Sentry `Transport`