Merge pull request #11 from Clearfacts/CLEARFACTS-8478

[CLEARFACTS-8478] start session after filling in server params
Clearfacts · Apr 5, 2023 · b19381e · b19381e
2 parents 67f3dbc + f8d6260
commit b19381e
Showing 1 changed file with 7 additions and 7 deletions.
diff --git a/src/Processor/SessionRequestProcessor.php b/src/Processor/SessionRequestProcessor.php
@@ -28,13 +28,6 @@ public function processRecord(array $record): array
             if ('cli' === PHP_SAPI) {
                 $this->sessionId = getmypid();
             } else {
-                try {
-                    $this->session->start();
-                    $this->sessionId = $this->session->getId();
-                } catch (\RuntimeException $e) {
-                    $this->sessionId = '????????';
-                }
-
                 $this->_server = [
                     'http.url' => (@$_SERVER['HTTP_HOST']) . '/' . (@$_SERVER['REQUEST_URI']),
                     'http.method' => @$_SERVER['REQUEST_METHOD'],
@@ -45,6 +38,13 @@ public function processRecord(array $record): array
 
                 $this->_post = $this->clean($_POST);
                 $this->_get = $this->clean($_GET);
+
+                try {
+                    $this->session->start();
+                    $this->sessionId = $this->session->getId();
+                } catch (\RuntimeException $e) {
+                    $this->sessionId = '????????';
+                }
             }
         }