Hookpoint for middleware checking is inadequate, causing succeeding i…

…nitializers to be executed after build_middleware_stack Fixes #3060
railsadminteam · Sep 23, 2018 · 94928d1 · 94928d1
1 parent 6d08234
commit 94928d1
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/lib/rails_admin/engine.rb b/lib/rails_admin/engine.rb
@@ -43,7 +43,8 @@ class Engine < Rails::Engine
       Dir[File.join(File.dirname(__FILE__), '../tasks/*.rake')].each { |f| load f }
     end
 
-    initializer 'RailsAdmin check for required middlewares', after: :build_middleware_stack do |app|
+    # Check for required middlewares, can be missing in Rails API mode
+    config.after_initialize do |app|
       has_session_store = app.config.middleware.to_a.any? { |m| m.klass.try(:<=, ActionDispatch::Session::AbstractStore) } || ::Rails.version < '5.0'
       loaded = app.config.middleware.to_a.map(&:name)
       required = %w(ActionDispatch::Cookies ActionDispatch::Flash Rack::MethodOverride)