Skip to content

Commit 4529ba5

Browse files
sentrivanaclaude
andcommitted
fix: Add breadcrumbs before early return in pymongo streaming path
Same issue as asyncpg/aiomysql — the no-current-span guard was skipping breadcrumb creation. Move breadcrumb and data computation before the guard. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 00edc42 commit 4529ba5

1 file changed

Lines changed: 7 additions & 7 deletions

File tree

sentry_sdk/integrations/pymongo.py

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -152,9 +152,6 @@ def started(self, event: "CommandStartedEvent") -> None:
152152
query = json.dumps(command, default=str)
153153

154154
if has_span_streaming_enabled(client.options):
155-
if sentry_sdk.traces.get_current_span() is None:
156-
return
157-
158155
span_first_data = {
159156
"db.operation.name": operation_name,
160157
"db.collection.name": collection_name,
@@ -164,10 +161,6 @@ def started(self, event: "CommandStartedEvent") -> None:
164161
**db_data,
165162
}
166163

167-
span = sentry_sdk.traces.start_span(
168-
name=query, attributes=span_first_data
169-
)
170-
171164
with capture_internal_exceptions():
172165
sentry_sdk.add_breadcrumb(
173166
message=query,
@@ -176,6 +169,13 @@ def started(self, event: "CommandStartedEvent") -> None:
176169
data=span_first_data,
177170
)
178171

172+
if sentry_sdk.traces.get_current_span() is None:
173+
return
174+
175+
span = sentry_sdk.traces.start_span(
176+
name=query, attributes=span_first_data
177+
)
178+
179179
else:
180180
tags = {
181181
"db.name": db_name,

0 commit comments

Comments
 (0)