Skip to content

Set alert_this_error when setting errors#259

Merged
unflxw merged 1 commit intomainfrom
set-error-alert-this-error
Feb 18, 2026
Merged

Set alert_this_error when setting errors#259
unflxw merged 1 commit intomainfrom
set-error-alert-this-error

Conversation

@unflxw
Copy link
Contributor

@unflxw unflxw commented Feb 17, 2026

When calling set_error to set an error on the current trace, set the appsignal.alert_this_error attribute on it as well.

This attribute has no effect when set_error is used by send_error or send_error_with_context, which produce single-span traces. The error in a root span will always be alerted.

This attribute has no effect when not using collector mode. The attribute is ignored by the agent.

When calling set_error on collector mode when the current span is some arbitrary span in the trace, this attribute ensures that the error (which was intentionally added to the span) is reported to AppSignal.

When calling `set_error` to set an error on the current trace, set
the `appsignal.alert_this_error` attribute on it as well.

This attribute has no effect when `set_error` is used by `send_error`
or `send_error_with_context`, which produce single-span traces. The
error in a root span will always be alerted.

This attribute has no effect when not using collector mode. The
attribute is ignored by the agent.

When calling `set_error` on collector mode when the current span is
some arbitrary span in the trace, this attribute ensures that the
error (which was intentionally added to the span) is reported
to AppSignal.
@unflxw unflxw requested review from lipskis and tombruijn February 17, 2026 13:39
@unflxw unflxw self-assigned this Feb 17, 2026
@tombruijn tombruijn added the enhancement An improvement to an existing feature. label Feb 17, 2026
@unflxw unflxw merged commit 28d3b5e into main Feb 18, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement An improvement to an existing feature.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants