aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib
diff options
context:
space:
mode:
authorAlberto Almagro <alberto.almagro@rakuten.com>2017-08-03 22:20:52 +0200
committerAlberto Almagro <alberto.almagro@rakuten.com>2017-08-14 23:56:13 +0200
commit7fade3dcbcfc1f5ba6c455c04f04040f077e0d70 (patch)
treec96468d7917bcf00c578af678c6f43f3037f5ab0 /actionpack/lib
parent10bf93ef92a70ae511036134290bf0e2de184b5c (diff)
downloadrails-7fade3dcbcfc1f5ba6c455c04f04040f077e0d70.tar.gz
rails-7fade3dcbcfc1f5ba6c455c04f04040f077e0d70.tar.bz2
rails-7fade3dcbcfc1f5ba6c455c04f04040f077e0d70.zip
Load Parameters configurations on :action_controller only once
Fixes regression ActionController::UnpermittedParameters not raised. The inner hook was being executed twice, once when ActionController::Base was loaded and again when ActionController::API was loaded. As options.delete operations inside the block are not idempotent, the second time it was run there was no configuration option available
Diffstat (limited to 'actionpack/lib')
-rw-r--r--actionpack/lib/action_controller/railtie.rb2
1 files changed, 1 insertions, 1 deletions
diff --git a/actionpack/lib/action_controller/railtie.rb b/actionpack/lib/action_controller/railtie.rb
index cc02c9a53a..769be39004 100644
--- a/actionpack/lib/action_controller/railtie.rb
+++ b/actionpack/lib/action_controller/railtie.rb
@@ -24,7 +24,7 @@ module ActionController
initializer "action_controller.parameters_config" do |app|
options = app.config.action_controller
- ActiveSupport.on_load(:action_controller) do
+ ActiveSupport.on_load(:action_controller, run_once: true) do
ActionController::Parameters.permit_all_parameters = options.delete(:permit_all_parameters) { false }
if app.config.action_controller[:always_permitted_parameters]
ActionController::Parameters.always_permitted_parameters =