The current redmine-3.0 branch requires signout_url and idp_slo_target_url but the callback path doesn't exist, SLS hasn't been implemented on the Redmine side in this branch. Looks like it was added for 4.x. Consequently, logout cannot succeed but you also cannot make Redmine use local logout (even though the code is actually there to do this).
So this PR does three things:
Stops signout_url and idp_slo_target_url being required for validation (SAML does not require them)
Fixes a little typo in the sign-out method which causes an error
Updates the sample documentation according to what I've discovered the settings should be
The current redmine-3.0 branch requires
signout_urlandidp_slo_target_urlbut the callback path doesn't exist, SLS hasn't been implemented on the Redmine side in this branch. Looks like it was added for 4.x. Consequently, logout cannot succeed but you also cannot make Redmine use local logout (even though the code is actually there to do this).So this PR does three things:
signout_urlandidp_slo_target_urlbeing required for validation (SAML does not require them)Hope this helps! :-)