mirror_php/moodle
1
0
Fork 0
mirror of https://github.com/moodle/moodle.git synced 2025-02-22 10:57:20 +01:00
Code Issues Projects Releases Wiki Activity

Merge branch 'MDL-83205-main' of https://github.com/andrewnicols/moodle

Browse Source
This commit is contained in:
Huong Nguyen 2024-09-27 11:26:42 +07:00
commit 955f04af77
No known key found for this signature in database
GPG Key ID: 40D88AB693A3E72A
1 changed files with 16 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
lib/classes/session/redis.php
View File

@ -167,12 +167,15 @@ class redis extends handler implements SessionHandlerInterface {
}
// This sets the Redis session lock expiry time to whatever is lower, either
// the PHP execution time `max_execution_time`, if the value was defined in
// the `php.ini` or the globally configured `sessiontimeout`. Setting it to
// the lower of the two will not make things worse it if the execution timeout
// the PHP execution time `max_execution_time`, if the value is positive, or the
// globally configured `sessiontimeout`.
//
// Setting it to the lower of the two will not make things worse it if the execution timeout
// is longer than the session timeout.
//
// For the PHP execution time, once the PHP execution time is over, we can be sure
// that the lock is no longer actively held so that the lock can expire safely.
//
// Although at `lib/classes/php_time_limit.php::raise(int)`, Moodle can
// progressively increase the maximum PHP execution time, this is limited to the
// `max_execution_time` value defined in the `php.ini`.
@ -180,9 +183,19 @@ class redis extends handler implements SessionHandlerInterface {
// once the session itself expires.
// If we unnecessarily hold the lock any longer, it blocks other session requests.
$this->lockexpire = ini_get('max_execution_time');
if ($this->lockexpire < 0) {
// If the max_execution_time is set to a value lower than 0, which is invalid, use the default value.
// https://www.php.net/manual/en/info.configuration.php#ini.max-execution-time defines the default as 30.
// Note: This value is not available programatically.
$this->lockexpire = 30;
}
if (empty($this->lockexpire) || ($this->lockexpire > (int)$CFG->sessiontimeout)) {
// The value of the max_execution_time is either unlimited (0), or higher than the session timeout.
// Cap it at the session timeout.
$this->lockexpire = (int)$CFG->sessiontimeout;
}
if (isset($CFG->session_redis_lock_expire)) {
$this->lockexpire = (int)$CFG->session_redis_lock_expire;
}