Merge pull request #5 from ElAberino/main

Fix constructor and method calls.
Treblle · Dec 6, 2022 · d68b1a2 · d68b1a2
2 parents 10e5862 + 3e7efde
commit d68b1a2
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 5 deletions.
diff --git a/src/DataProvider.php b/src/DataProvider.php
@@ -41,9 +41,20 @@ public static function getSubscribedEvents(): array
 
     public function onKernelRequest(RequestEvent $event): void
     {
-        if ($event->isMasterRequest()) {
-            $this->httpRequest = $event->getRequest();
-            $this->timestampStart = microtime(true);
+        if (method_exists($event, 'isMainRequest')) { // Symfony >= 5.3
+            if ($event->isMainRequest()) {
+                $this->httpRequest = $event->getRequest();
+                $this->timestampStart = microtime(true);
+            }
+
+            return;
+        }
+
+        if (method_exists($event, 'isMasterRequest')) { // Symfony < 5.3
+            if ($event->isMasterRequest()) {
+                $this->httpRequest = $event->getRequest();
+                $this->timestampStart = microtime(true);
+            }
         }
     }
 

diff --git a/src/DependencyInjection/TreblleFactory.php b/src/DependencyInjection/TreblleFactory.php
@@ -25,7 +25,6 @@ public static function createTreblle(
         ErrorDataProvider $errorDataProvider
     ): Treblle {
         return new Treblle(
-            $configuration->getEndpointUrl(),
             $configuration->getApiKey(),
             $configuration->getProjectId(),
             $client,

diff --git a/tests/TreblleIntegrationTest.php b/tests/TreblleIntegrationTest.php
@@ -79,7 +79,6 @@ protected function setUp(): void
         $this->dataProvider = new DataProvider(new PayloadAnonymizer([]));
 
         $this->treblle = new Treblle(
-            'http://127.0.0.1',
             'my api key',
             'my project id',
             $client,