Problem/Motivation
When using PHP 7.1 and Memcache, the 1.14 version of the simplesamlphp library fails. This is due to the check done for memcache not being compatible with the name of the extension for PHP 7.1.
This issue has been discussed and fixed in the library but has been committed only to the 1.15 branch. This branch also includes changes in the Main class name, which could be handled by autoload.php if composer was used for updating the library to 1.15.
https://github.com/simplesamlphp/simplesamlphp/issues/715
This is being used by several enterprise customers and is their only block to use PHP 7.1.
Proposed resolution
- Allow the module's composer.json to update to the latest SimpleSAMLPHP library V.1.15 or any of its RCs.
- Check that there are no other compatibility issues with 1.15.
Remaining tasks
- Patch composer.json
- test working authentication.
Comment | File | Size | Author |
---|---|---|---|
#2 | php71-memcache-2925106-2.patch | 279 bytes | lcatlett |
Comments
Comment #2
lcatlett CreditAttribution: lcatlett commentedPatch adding support for memcache storage for php71 attached
Comment #3
dakku CreditAttribution: dakku commentedHi Lindsey, long time no see. Thank you for the patch.
We have a number of open threads regarding updating the lib version. I havent had time to test the lib upgrade yet. I am assuming this has been tested already on Acquia Client sites?
Please see ongoing discussions here:
https://www.drupal.org/project/simplesamlphp_auth/issues/2929202
Comment #4
dakku CreditAttribution: dakku commentedComment #5
dakku CreditAttribution: dakku commentedHI @lcatlett, Do you mind if we close this thread and move the discussion to:
https://www.drupal.org/project/simplesamlphp_auth/issues/2929202
We have quite a few threads open on this but the message seems somewhat mixed.
Comment #6
lcatlett CreditAttribution: lcatlett commented@dakku I posted an updated patch at https://www.drupal.org/project/simplesamlphp_auth/issues/2929202#comment... so this thread can be closed.