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.

CommentFileSizeAuthor
#2 php71-memcache-2925106-2.patch279 byteslcatlett
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

camoa created an issue. See original summary.

lcatlett’s picture

FileSize
279 bytes

Patch adding support for memcache storage for php71 attached

dakku’s picture

Hi 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

dakku’s picture

dakku’s picture

Status: Active » Closed (duplicate)

HI @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.

lcatlett’s picture

@dakku I posted an updated patch at https://www.drupal.org/project/simplesamlphp_auth/issues/2929202#comment... so this thread can be closed.