fix(core): Wrap beforeSendLog in consoleSandbox (#16968)

resolves #16920

This will prevent recursive calls to `beforeSendLog` when using console
logging instrumentation which will cause crashes.

Author: AbhiPrasad

packages/core/src/logs/exports.ts

@@ -7,7 +7,7 @@ import type { Scope, ScopeData } from '../scope';
 import type { Log, SerializedLog, SerializedLogAttributeValue } from '../types-hoist/log';
 import { mergeScopeData } from '../utils/applyScopeDataToEvent';
 import { isParameterizedString } from '../utils/is';
-import { debug } from '../utils/logger';
+import { consoleSandbox, debug } from '../utils/logger';
 import { _getSpanForScope } from '../utils/spanOnScope';
 import { timestampInSeconds } from '../utils/time';
 import { SEVERITY_TEXT_TO_SEVERITY_NUMBER } from './constants';
@@ -168,7 +168,8 @@ export function _INTERNAL_captureLog(
 
   client.emit('beforeCaptureLog', processedLog);
 
-  const log = beforeSendLog ? beforeSendLog(processedLog) : processedLog;
+  // We need to wrap this in `consoleSandbox` to avoid recursive calls to `beforeSendLog`
+  const log = beforeSendLog ? consoleSandbox(() => beforeSendLog(processedLog)) : processedLog;
   if (!log) {
     client.recordDroppedEvent('before_send', 'log_item', 1);
     DEBUG_BUILD && debug.warn('beforeSendLog returned null, log will not be captured.');