Merge pull request #1446 from stakovicz/feat-1415-session-in-db

#1415 Session in DB
afup · Feb 12, 2024 · ff5055c · ff5055c
2 parents 23d7eac + ae21a63
commit ff5055c
Show file tree

Hide file tree

Showing 6 changed files with 31 additions and 13 deletions.
diff --git a/app/config/config.yml b/app/config/config.yml
@@ -332,9 +332,8 @@ framework:
     default_locale:  "%locale%"
     trusted_hosts:   ~
     session:
-        # http://symfony.com/doc/current/reference/configuration/framework.html#handler-id
-        storage_id: session.storage.php_bridge
-        handler_id: ~
+        # https://symfony.com/doc/4.x/session/database.html#store-sessions-in-a-relational-database-mariadb-mysql-postgresql
+        handler_id: Symfony\Component\HttpFoundation\Session\Storage\Handler\PdoSessionHandler
     fragments:       ~
     http_method_override: true
     assets: ~

diff --git a/app/config/services.yml b/app/config/services.yml
@@ -14,6 +14,15 @@ services:
     _defaults:
         public: true
 
+    Symfony\Component\HttpFoundation\Session\Storage\Handler\PdoSessionHandler:
+        public:    false
+        arguments:
+            - 'mysql:host=%database_host%;port=%database_port%;dbname=%database_name%'
+            - db_username: '%database_user%'
+              db_password: '%database_password%'
+              lock_mode: !php/const:Symfony\Component\HttpFoundation\Session\Storage\Handler\PdoSessionHandler::LOCK_NONE
+
+
     AppBundle\Controller\Event\:
         resource: '../../sources/AppBundle/Controller/Event/*'
         autowire: true

diff --git a/db/migrations/20240106213200_sessions.php b/db/migrations/20240106213200_sessions.php
@@ -0,0 +1,20 @@
+<?php
+
+use Phinx\Migration\AbstractMigration;
+
+class Sessions extends AbstractMigration
+{
+    public function change()
+    {
+        $sql = <<<EOF
+CREATE TABLE `sessions` (
+    `sess_id` VARBINARY(128) NOT NULL PRIMARY KEY,
+    `sess_data` BLOB NOT NULL,
+    `sess_lifetime` INTEGER UNSIGNED NOT NULL,
+    `sess_time` INTEGER UNSIGNED NOT NULL,
+    INDEX `sessions_sess_lifetime_idx` (`sess_lifetime`)
+) COLLATE utf8mb4_bin, ENGINE = InnoDB;
+EOF;
+        $this->execute($sql);
+    }
+}
diff --git a/htdocs/app.php b/htdocs/app.php
@@ -19,8 +19,6 @@
         exit('You are not allowed to access this file. Check '.basename(__FILE__).' for more information.');
     }
 
-    session_start();
-
     /** @var \Composer\Autoload\ClassLoader $loader */
     $loader = require __DIR__.'/../vendor/autoload.php';
     Debug::enable();
@@ -34,8 +32,6 @@
     /** @var \Composer\Autoload\ClassLoader $loader */
     $loader = require __DIR__.'/../vendor/autoload.php';
 
-    session_start();
-
     $kernel = new AppKernel('prod', false);
 }
 

diff --git a/htdocs/include/prepend.inc.php b/htdocs/include/prepend.inc.php
@@ -3,8 +3,6 @@
 // Initialisation
 ob_start();
 
-session_start();
-
 // Inclusion de l'autoload de composer
 require_once dirname(__FILE__) . '/../../vendor/autoload.php';
 

diff --git a/htdocs/pages/planete/app.php b/htdocs/pages/planete/app.php
@@ -28,8 +28,6 @@
         exit('You are not allowed to access this file. Check '.basename(__FILE__).' for more information.');
     }
 
-    session_start();
-
     /** @var ClassLoader $loader */
     $loader = require __DIR__.'/../../../vendor/autoload.php';
     Debug::enable();
@@ -43,8 +41,6 @@
     /** @var ClassLoader $loader */
     $loader = require __DIR__.'/../../../vendor/autoload.php';
 
-    session_start();
-
     $kernel = new PlaneteAppKernel('prod', false);
 }