-
Notifications
You must be signed in to change notification settings - Fork 141
Open
Description
Latest release 2025040400 is giving a 500 error if redis is activated with $CFG->auth_saml2_store = '\auth_saml2\redis_store'; and $CFG->auth_saml2_redis_server = '127.0.0.1';
Underlying error appears to be:
[Sun Apr 13 01:59:48.439402 2025] [php:error] [pid 56771:tid 56771] [client 2a02:x:x:x:x:x:x:x:x] PHP Fatal error: Declaration of auth_saml2\\redis_store::get($type, $key) must be compatible with SimpleSAML\\Store\\StoreInterface::get(string $type, string $key): mixed in /var/www/html/m45yellow.hashbed600.co.uk/auth/saml2/classes/redis_store.php on line 84, referer: https://example.moodle.foo/admin/index.php?cache=1
FWIW adding the type declarations and return type at line 84 of redis_store.php seems to clear the issue.
Metadata
Metadata
Assignees
Labels
No labels