moodle-auth_saml2 icon indicating copy to clipboard operation
moodle-auth_saml2 copied to clipboard

set SESSION->saml2idp during unsolicited logins

Open skodak opened this issue 2 years ago • 3 comments

This makes the IdP known in saml_login_complete() and tenant callback.

skodak avatar May 03 '23 19:05 skodak

I suppose this might affect Moodle for Workplace too because the callback for tenant support with $SESSION->saml2idp originated there

skodak avatar May 03 '23 19:05 skodak

thanks @skodak - @kabalin are you able to review this in the context of workplace?

brendanheywood avatar May 10 '23 02:05 brendanheywood

thanks @skodak - @kabalin are you able to review this in the context of workplace?

This change does not affect Workplace (tested it thoroughly). Cleared to land :)

kabalin avatar May 15 '23 16:05 kabalin

any news?

skodak avatar Jun 12 '24 09:06 skodak

looks like we forgot about this one - happy to merge, but someone needs to update the patch for the new version of simplesamlphp - I think the change needs to go here now instead: https://github.com/catalyst/moodle-auth_saml2/blob/MOODLE_39_STABLE/.extlib/simplesamlphp/modules/saml/src/Auth/Source/SP.php#L1136

danmarsden avatar Jun 13 '24 06:06 danmarsden

patch updated

skodak avatar Jun 13 '24 06:06 skodak