Skip to content

.

0bf02b4
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Merged

ref(rust): Don't implement separate scope management #5639

.
0bf02b4
Select commit
Loading
Failed to load commit list.
GitHub Actions / warden completed Mar 11, 2026 in 1m 43s

2 issues

High

sentry_span.__exit__() called without required arguments causes TypeError - `sentry_sdk/integrations/rust_tracing.py:226`

The __exit__() method is called without arguments, but Python's context manager protocol requires three arguments (exc_type, exc_value, traceback). The Span.__exit__ method signature is def __exit__(self, ty, value, tb) with no default values. This will raise TypeError: __exit__() missing 3 required positional arguments when on_close() is called.

Span.__exit__() called without required arguments causing TypeError at runtime - `sentry_sdk/integrations/rust_tracing.py:226`

The on_close method calls sentry_span.__exit__() without any arguments, but Span.__exit__ requires three positional arguments (ty, value, tb) as shown in sentry_sdk/tracing.py line 396-398. This will cause a TypeError at runtime when closing any Rust tracing span, completely breaking the integration's span lifecycle.

4 skills analyzed
Skill Findings Duration Cost
code-review 1 1m 3s $0.27
find-bugs 1 1m 33s $0.55
skill-scanner 0 33.1s $0.17
security-review 0 23.1s $0.13

Duration: 3m 32s · Tokens: 585.0k in / 6.5k out · Cost: $1.13 (+fix_gate: $0.01, +dedup: $0.00, +extraction: $0.00)