devise_saml_authenticatable icon indicating copy to clipboard operation
devise_saml_authenticatable copied to clipboard
verified publisher icon apokalipto

Fix prepend_before_action order on SamlSessionsController

Open anna-dev opened this issue 1 year ago • 1 comments

Fixes issue 219: https://github.com/apokalipto/devise_saml_authenticatable/issues/219

The names in prepend_before_action do have the wrong order. The last one gets called first. If store_info_for_sp_initiated_logout gets called before verify_signed_out_user, this lead to the issuer that current_user is nil.

anna-dev avatar Nov 28 '24 16:11 anna-dev

Arguably, store_info_for_sp_initiated_logout should also terminate if current_user.nil?

doconnor-clintel avatar Feb 23 '25 00:02 doconnor-clintel