SimpleSAML_Session::getAuthority() has been removed in 1.14, and this causes /user/logout to fail. I've provided a patch.

CommentFileSizeAuthor
#2 drupalauth4ssp-getAuthority-2691915-2.patch970 bytessanchiz
expire-authorities.patch619 bytesagileware

Comments

Agileware created an issue. See original summary.

sanchiz’s picture

sanchiz commented
Status: Active » Needs review
StatusFileSize
new drupalauth4ssp-getAuthority-2691915-2.patch970 bytes

Thank you, Agileware! Committed.

I've also update your patch to keep backward compatibility.

Available now in 7.x-1.x-dev or in 7.x-1.0-beta3 and later once released.

sanchiz’s picture

sanchiz commented
Status: Needs review » Fixed

  • sanchiz committed a6f652f on 7.x-1.x authored by Agileware 
    Issue #2691915 by Agileware: Removal of SimpleSAML_Session::getAuthority...

  • sanchiz committed c4e21ba on 7.x-1.x 
    Issue #2691915: Add backward compatibility
sanchiz’s picture

sanchiz commented
Status: Fixed » Reviewed & tested by the community
sanchiz’s picture

sanchiz commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

geekwisdom’s picture

geekwisdom commented

It seems this is not bullet proof enough.

if (method_exists($session, 'setAuthorityExpire')) {

should be

if (method_exists($session, 'setAuthorityExpire') && method_exists($session, 'getAuthority')) {

or perhaps just

if (method_exists($session, 'getAuthority')) {