Fix / Register tracing context manually

looplab / eventhorizon

Event Sourcing for Go!

Apache License 2.0

1.57k stars 196 forks source link

Fix / Register tracing context manually #372

Closed maxekman closed 2 years ago

maxekman commented 2 years ago

Description

This should enable Elastic APM to register tracing of events in outbox and event bus handlers.

NOTE: Users of Jaeger will need to call tracing.RegisterContext() manually.

The reason for adding this is described in this issue: https://github.com/elastic/apm/issues/122

Affected Components

Tracing

Related Issues

Solution and Design

Steps to test and verify

coveralls commented 2 years ago

Coverage decreased (-0.08%) to 68.787% when pulling a7ea4e58cef2503e1de269e3b4bf3b74ac8a42a1 on maxekman:fix-optional-tracing-context into 1b41069a35e845ea6d13d31341011a760dd8ef86 on looplab:main.