Browse Source

Devise logout via get so we don't rely on js

pull/392/head
baarkerlounger 3 years ago
parent
commit
f550f46452
  1. 2
      app/views/layouts/application.html.erb
  2. 2
      config/initializers/devise.rb
  3. 2
      config/routes.rb

2
app/views/layouts/application.html.erb

@ -46,7 +46,7 @@
component.navigation_item(text: 'Logs', href: case_logs_path)
component.navigation_item(text: 'Your organisation', href: "/organisations/#{current_user.organisation.id}")
component.navigation_item(text: 'Your account', href: user_path(current_user))
component.navigation_item(text: 'Sign out', href: destroy_user_session_path, options: { data: {turbo_method: :delete}, method: :delete })
component.navigation_item(text: 'Sign out', href: destroy_user_session_path)
end
end
%>

2
config/initializers/devise.rb

@ -267,7 +267,7 @@ Devise.setup do |config|
# config.navigational_formats = ['*/*', :html]
# The default HTTP method used to sign out a resource. Default is :delete.
config.sign_out_via = :delete
config.sign_out_via = :get
# ==> OmniAuth
# Add a new OmniAuth provider. Check the wiki for more information on setting

2
config/routes.rb

@ -11,7 +11,7 @@ Rails.application.routes.draw do
two_factor_authentication: "auth/two_factor_authentication",
},
path_names: { sign_in: "sign-in", sign_out: "sign-out", two_factor_authentication: "two-factor-authentication" },
sign_out_via: %i[delete get],
sign_out_via: %i[get],
}
devise_scope :admin_user do

Loading…
Cancel
Save