afup#1415 Session in DB

stakovicz · Jan 6, 2024 · 9e33fec · 9e33fec
1 parent 82d4e55
commit 9e33fec
Show file tree

Hide file tree

Showing 9 changed files with 36 additions and 18 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,12 @@ 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%' }
+
     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);
 }
 

diff --git a/planete/app/config/config.yml b/planete/app/config/config.yml
@@ -19,8 +19,8 @@ framework:
     default_locale:  "%locale%"
     trusted_hosts:   ~
     session:
-        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/planete/app/config/services.yml b/planete/app/config/services.yml
@@ -4,6 +4,12 @@ services:
         autowire: true
         autoconfigure: 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%' }
+
     AppBundle\LegacyModelFactory: ~
 
     PlanetePHP\:

diff --git a/sources/Afup/Bootstrap/Http.php b/sources/Afup/Bootstrap/Http.php
@@ -26,9 +26,6 @@
 if (ob_get_level() === 0) {
     ob_start();
 }
-if (session_status() === PHP_SESSION_NONE) {
-    session_start();
-}
 
 // mise à jour des paramètrage PHP en fonction de la configuration