The _sentry_execute_command function manually enters db_span and cache_span contexts (lines 126 and 145) but doesn't use try/finally to ensure __exit__ is called on exceptions. If await old_execute_command() raises an exception, the spans' __exit__ methods are never called, causing span leaks and potentially corrupting scope state. The synchronous version in _sync_common.py correctly uses try/finally for this pattern.

In sentry_patched_execute_command, when old_execute_command raises an exception, the value variable is never assigned. The finally block then attempts to use value in _set_cache_data(cache_span, self, cache_properties, value), causing a NameError. While capture_internal_exceptions() catches this, it results in unnecessary internal exception noise and obscures the real error from monitoring.

The async _sentry_execute_command function does not wrap the Redis command execution in a try/finally block. When old_execute_command raises an exception, db_span.__exit__() and cache_span.__exit__() are never called, leaving spans unclosed. The sync version in _sync_common.py correctly uses try/finally (lines 151-160).

Also found at:

• sentry_sdk/integrations/redis/_async_common.py:147-148

In sentry_patched_execute_command, the finally block calls _set_cache_data(cache_span, self, cache_properties, value) which references value. However, value is only assigned if old_execute_command succeeds. If old_execute_command raises an exception, value is undefined, causing an UnboundLocalError in the finally block. While capture_internal_exceptions() wraps this call and will suppress the error, this still creates an unnecessary exception that gets silently caught, and the span cleanup logic after it (cache_span.__exit__) will still execute successfully.

The new StreamedSpan code path in _set_cache_data lacks dedicated test coverage. Existing Redis tests do not include tests with _experiments={"trace_lifecycle": "stream"} enabled. While the implementation correctly uses isinstance checks to differentiate span types and both set_attribute and set_data have compatible signatures, the new streaming path is untested.