diff --git a/app/controllers/providers/omniauth_callbacks_controller.rb b/app/controllers/providers/omniauth_callbacks_controller.rb index 322d29fbc..b7811dd3d 100644 --- a/app/controllers/providers/omniauth_callbacks_controller.rb +++ b/app/controllers/providers/omniauth_callbacks_controller.rb @@ -16,14 +16,6 @@ def saml provider = Provider.create! email: info_hash_email, first_office_code: info_hash_office_code end - # reset the session on login, otherwise the session expires after 14 days - # and logouts crash because there is no session data (SAML_UID) to logout with - # This may need to change if we want to preserve check data over a login - # https://stackoverflow.com/questions/4812813/rails-login-reset-session - # - # Some docs suggest that devise maybe doing this behind the scenes, so - # this might be an investigation if changing this doesn't work. - reset_session # Portal has checked that we have the correct role, so we can just sign in sign_in_and_redirect provider, event: :authentication end