Set alert_this_error when setting errors

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.

Author: unflxw

.changesets/fix-an-issue-where-errors-set-with-the--set_error--helper-would-not-always-be-reported-as-errors-in-appsignal-when-using-collector-mode-.md

@@ -0,0 +1,6 @@
+---
+bump: patch
+type: fix
+---
+
+Fix an issue where errors set with the `set_error` helper would not always be reported as errors in AppSignal when using collector mode.

src/appsignal/tracing.py

@@ -91,7 +91,7 @@ def set_root_name(root_name: str, span: Span | None = None) -> None:
 def set_error(error: Exception, span: Span | None = None) -> None:
     span = span or trace.get_current_span()
     span.set_status(Status(StatusCode.ERROR))
-    span.record_exception(error)
+    span.record_exception(error, attributes={"appsignal.alert_this_error": True})
 
 
 def send_error(error: Exception) -> None: