Skip to content

Commit

Permalink
Merge branch 'QA_5_1'
Browse files Browse the repository at this point in the history
Signed-off-by: William Desportes <williamdes@wdes.fr>
  • Loading branch information
williamdes committed Jul 22, 2021
2 parents 7e567be + be64a4f commit 5b8cd0f
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 2 deletions.
2 changes: 1 addition & 1 deletion libraries/classes/Session.php
Expand Up @@ -183,7 +183,7 @@ public static function setUp(Config $config, ErrorHandler $errorHandler)
ini_set('session.cookie_httponly', '1');
if (PHP_VERSION_ID >= 70300) {
// add SameSite to the session cookie
ini_set('session.cookie_samesite', $config->get('CookieSameSite'));
ini_set('session.cookie_samesite', $config->get('CookieSameSite') ?? '');
}

// do not force transparent session ids
Expand Down
7 changes: 6 additions & 1 deletion libraries/config.default.php
Expand Up @@ -786,7 +786,12 @@
/**
* sets SameSite attribute of the Set-Cookie HTTP response header
*
* @global boolean $cfg['CookieSameSite']
* Valid values are:
* - Lax
* - Strict
* - None
*
* @global string $cfg['CookieSameSite']
*/
$cfg['CookieSameSite'] = 'Strict';

Expand Down

0 comments on commit 5b8cd0f

Please sign in to comment.