Merge pull request #26 from code-tool/fix-listeners-order

Fix symfony.start span pos by changing eventListeners order

Author: allflame

src/Bridge/AppStartSpanListener.php

@@ -22,21 +22,21 @@ public function __construct(TracerInterface $tracer)
 
     public static function getSubscribedEvents(): array
     {
-        return [RequestEvent::class => ['onRequest', -1],];
+        return [RequestEvent::class => ['onRequest', -1023],];
     }
 
     public function onRequest(RequestEvent $event)
     {
-        $request = $event->getRequest();
         if (false === $this->isMainRequestEvent($event)) {
             return $this;
         }
+
         $this->tracer
             ->start('symfony.start')
             ->addTag(new SpanKindServerTag())
             ->addTag(new SymfonyComponentTag())
             ->addTag(new SymfonyVersionTag())
-            ->start((int)(1000000 * $request->server->get('REQUEST_TIME_FLOAT', microtime(true))))
+            ->start((int)(1000000 * $event->getRequest()->server->get('REQUEST_TIME_FLOAT', microtime(true))))
             ->finish();
 
         return $this;

src/Bridge/GlobalSpanListener.php

@@ -20,7 +20,7 @@ public function __construct(GlobalSpanHandler $handler)
     public static function getSubscribedEvents(): array
     {
         return [
-            RequestEvent::class => ['onRequest', 30],
+            RequestEvent::class => ['onRequest', 25],
             TerminateEvent::class => ['onTerminate', 4096],
         ];
     }

src/Bridge/RequestSpanListener.php

@@ -37,7 +37,7 @@ public function __construct(\SplStack $stack, NameGeneratorInterface $nameGenera
     public static function getSubscribedEvents(): array
     {
         return [
-            RequestEvent::class => ['onRequest', 29],
+            RequestEvent::class => ['onRequest', -1024],
             ResponseEvent::class => ['onResponse', -1024],
             ExceptionEvent::class => ['onKernelException', 0],
         ];

src/Name/Generator/ControllerNameGenerator.php

@@ -15,7 +15,7 @@ public static function getSubscribedEvents(): array
     {
         return [
             // Subscribe after route was resolved and request attributes were set
-            RequestEvent::class => ['onRequest', 31],
+            RequestEvent::class => ['onRequest', 30],
             TerminateEvent::class => ['onTerminate', -16384],
         ];
     }

src/Name/Generator/DefaultNameGenerator.php

@@ -17,7 +17,7 @@ public static function getSubscribedEvents(): array
     {
         return [
             // Subscribe after route was resolved and request attributes were set
-            RequestEvent::class => ['onRequest', 31],
+            RequestEvent::class => ['onRequest', 30],
             ConsoleCommandEvent::class => ['onCommand', 31],
             TerminateEvent::class => ['onTerminate', -16384],
             ConsoleTerminateEvent::class => ['onTerminate'],

src/Name/Generator/RequestNameGenerator.php

@@ -35,7 +35,7 @@ public static function getSubscribedEvents(): array
     {
         return [
             // Subscribe after route was resolved and request attributes were set
-            RequestEvent::class => ['onRequest', 31],
+            RequestEvent::class => ['onRequest', 30],
             TerminateEvent::class => ['onTerminate', -16384],
         ];
     }