catalyst / moodle-auth_saml2

SAML done 100% in Moodle, fast, simple, secure
https://moodle.org/plugins/auth_saml2
70 stars 132 forks source link

Support multiple idp's with the same metadata URL #799

Open Anders-Oerbaek-Pedersen opened 4 months ago

Anders-Oerbaek-Pedersen commented 4 months ago

Trying to solve: #796

brendanheywood commented 4 months ago

I haven't looked at this properly, but one thing we need to make sure doesn't happen is that people's hard coded links from say their portal into moodle which embed query params of an idp do not need to change. It looks to me like this is a breaking change, so maybe the way forward is for the ?idp=xxx param to accept the old and new way so its backwards compatible