Skip to content

Commit d01c306

Browse files
authored
Merge pull request #28 from grynchuk/master
Add trace reset on request terminate
2 parents 7084676 + 5bc7f72 commit d01c306

File tree

2 files changed

+6
-0
lines changed

2 files changed

+6
-0
lines changed

src/Bridge/TracerBridge.php

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
namespace Jaeger\Symfony\Bridge;
55

66
use Jaeger\Tracer\FlushableInterface;
7+
use Jaeger\Tracer\ResettableInterface;
78
use Symfony\Component\Console\Event\ConsoleTerminateEvent;
89
use Symfony\Component\EventDispatcher\EventSubscriberInterface;
910
use Symfony\Component\HttpKernel\Event\TerminateEvent;
@@ -28,5 +29,8 @@ public static function getSubscribedEvents(): array
2829
public function onTerminate(): void
2930
{
3031
$this->tracer->flush();
32+
if ($this->tracer instanceof ResettableInterface) {
33+
$this->tracer->reset();
34+
}
3135
}
3236
}

src/Resources/services.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -81,6 +81,8 @@ services:
8181
jaeger.tracer:
8282
class: Jaeger\Tracer\Tracer
8383
parent: jaeger.tracer.abstract
84+
tags:
85+
- { name: 'kernel.reset', method: 'reset' }
8486
jaeger.debuggable: '@jaeger.tracer'
8587
jaeger.flushable: '@jaeger.tracer'
8688
jaeger.context.extractor.env:

0 commit comments

Comments
 (0)