Skip to content

Commit d1de481

Browse files
authored
Make span.finish work (#3559)
1 parent 13441e3 commit d1de481

File tree

1 file changed

+11
-4
lines changed

1 file changed

+11
-4
lines changed

sentry_sdk/tracing.py

Lines changed: 11 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1268,7 +1268,7 @@ def __enter__(self):
12681268

12691269
def __exit__(self, ty, value, tb):
12701270
# type: (Optional[Any], Optional[Any], Optional[Any]) -> None
1271-
self._otel_span.end()
1271+
self.finish()
12721272
# XXX set status to error if unset and an exception occurred?
12731273
context.detach(self._ctx_token)
12741274

@@ -1524,9 +1524,16 @@ def is_success(self):
15241524
# type: () -> bool
15251525
return self._otel_span.status.code == StatusCode.OK
15261526

1527-
def finish(self, scope=None, end_timestamp=None):
1528-
# type: (Optional[sentry_sdk.Scope], Optional[Union[float, datetime]]) -> Optional[str]
1529-
pass
1527+
def finish(self, end_timestamp=None):
1528+
# type: (Optional[Union[float, datetime]]) -> Optional[str]
1529+
if end_timestamp is not None:
1530+
from sentry_sdk.integrations.opentelemetry.utils import (
1531+
convert_to_otel_timestamp,
1532+
)
1533+
1534+
self._otel_span.end(convert_to_otel_timestamp(end_timestamp))
1535+
else:
1536+
self._otel_span.end()
15301537

15311538
def to_json(self):
15321539
# type: () -> dict[str, Any]

0 commit comments

Comments
 (0)