Problem: SAML Auth PHP version 2 requires the PHP mcrypt extension which is unsupported.

Solution: use version 3 of library which has otherwise minimal changes.

CommentFileSizeAuthor
#2 samlauth-v3-3044116.patch1.3 KBklausi

Comments

klausi created an issue. See original summary.

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
commented
Status: Active » Needs review
StatusFileSize
new samlauth-v3-3044116.patch1.3 KB

Patch.

smfsh’s picture

smfsh commented
Assigned: Unassigned » smfsh
Status: Needs review » Reviewed & tested by the community

I've been wanting to do this myself for a while (because mcrypt is garbage) but was worried about the overhead in changes. I haven't taken the time to understand exactly whats different in the 3.0 library. Obviously the D7 version of samlauth is simpler than the D8 version, so maybe these considerations are all it requires. I'll release this into dev channel for now and then promote after a bit more testing and bug squashing if necessary.

Thanks for taking the leap for me!

  • smfsh committed 70127d8 on 7.x-1.x authored by klausi 
    Issue #3044116 by klausi: Update SAML Auth library to version 3 for PHP...
joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Status: Reviewed & tested by the community » Fixed

This appears to be committed, marking as fixed.

Status: Fixed » Closed (fixed)

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