Fix handling ESI request

getsentry · May 16, 2019 · c92cfe6 · c92cfe6
1 parent 02a05a0
commit c92cfe6
Show file tree

Hide file tree

Showing 2 changed files with 26 additions and 0 deletions.
diff --git a/src/EventListener/RequestListener.php b/src/EventListener/RequestListener.php
@@ -88,6 +88,10 @@ public function onKernelController(FilterControllerEvent $event): void
             return;
         }
 
+        if (! $event->getRequest()->attributes->has('_route')) {
+            return;
+        }
+
         $matchedRoute = $event->getRequest()->attributes->get('_route');
 
         Hub::getCurrent()

diff --git a/test/EventListener/RequestListenerTest.php b/test/EventListener/RequestListenerTest.php
@@ -355,6 +355,28 @@ public function testOnKernelControllerAddsRouteTag(): void
         $this->assertSame(['route' => 'sf-route'], $this->currentScope->getTags());
     }
 
+    public function testOnKernelControllerRouteTagIsNotSetIfRequestDoesNotHaveARoute(): void
+    {
+        $this->currentHub->configureScope(Argument::type('callable'))
+            ->shouldNotBeCalled();
+
+        $request = new Request();
+        $event = $this->prophesize(FilterControllerEvent::class);
+
+        $event->isMasterRequest()
+            ->willReturn(true);
+        $event->getRequest()
+            ->willReturn($request);
+
+        $listener = new RequestListener(
+            $this->currentHub->reveal(),
+            $this->prophesize(TokenStorageInterface::class)->reveal(),
+            $this->prophesize(AuthorizationCheckerInterface::class)->reveal()
+        );
+
+        $listener->onKernelController($event->reveal());
+    }
+
     public function testOnKernelRequestUserDataAndTagsAreNotSetInSubRequest(): void
     {
         $this->currentHub->configureScope(Argument::type('callable'))