fix(anthropic): Stop setting transaction status when child span fails by alexander-alderman-webb · Pull Request #5717 · getsentry/sentry-python · GitHub
Skip to content

fix(anthropic): Stop setting transaction status when child span fails - #5717

Merged
alexander-alderman-webb merged 12 commits into
masterfrom
webb/anthropic/remove-set-span-errored
Apr 9, 2026
Merged

alexander-alderman-webb merged 12 commits into
masterfrom
webb/anthropic/remove-set-span-errored

Conversation

@alexander-alderman-webb

@alexander-alderman-webb alexander-alderman-webb commented Mar 20, 2026

Copy link
Copy Markdown
Contributor

Description

Stop modifying the transaction, since the transaction may not be managed by anthropic.

Issues

Reminders

@github-actions

github-actions Bot commented Mar 20, 2026

Copy link
Copy Markdown
Contributor

@github-actions

github-actions Bot commented Apr 7, 2026

Copy link
Copy Markdown
Contributor

@alexander-alderman-webb
alexander-alderman-webb marked this pull request as ready for review April 7, 2026 13:38
@alexander-alderman-webb
alexander-alderman-webb requested a review from a team as a code owner April 7, 2026 13:38
Base automatically changed from webb/anthropic/remove-get-current-span to master April 9, 2026 07:46
@alexander-alderman-webb alexander-alderman-webb changed the title fix(anthropic): Stop setting transaction to errored when child span fails fix(anthropic): Stop setting transaction status when child span fails Apr 9, 2026
@alexander-alderman-webb
alexander-alderman-webb merged commit 0b990da into master Apr 9, 2026
158 checks passed
@alexander-alderman-webb
alexander-alderman-webb deleted the webb/anthropic/remove-set-span-errored branch April 9, 2026 07:59
mgaligniana pushed a commit to mgaligniana/sentry-python that referenced this pull request Aug 9, 2026
…getsentry#5717)

Stop setting the transaction status when an exception is raised, since the transaction may not be managed by the Anthropic integration.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants