diff options
author | Ryan Davis <ryand-ruby@zenspider.com> | 2014-06-03 16:57:22 -0700 |
---|---|---|
committer | Ryan Davis <ryand-ruby@zenspider.com> | 2014-06-03 16:57:22 -0700 |
commit | e63be2769c039e4e9ada523a8497ce3206cc8a9b (patch) | |
tree | f7fd2d05e8a02d259cae5a8a777fde5b57b09531 /actionpack/test/controller | |
parent | 6daa46621b7145fcf948f48bed01d3cfe9221613 (diff) | |
download | rails-e63be2769c039e4e9ada523a8497ce3206cc8a9b.tar.gz rails-e63be2769c039e4e9ada523a8497ce3206cc8a9b.tar.bz2 rails-e63be2769c039e4e9ada523a8497ce3206cc8a9b.zip |
Convert StrongParameters cache to a hash. This fixes an unbounded
memory leak demonstrated on @tenderlove's latest blog post:
http://tenderlovemaking.com/2014/06/02/yagni-methods-are-killing-me.html
Diffstat (limited to 'actionpack/test/controller')
-rw-r--r-- | actionpack/test/controller/parameters/parameters_permit_test.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/actionpack/test/controller/parameters/parameters_permit_test.rb b/actionpack/test/controller/parameters/parameters_permit_test.rb index 33a91d72d9..1856ecd42b 100644 --- a/actionpack/test/controller/parameters/parameters_permit_test.rb +++ b/actionpack/test/controller/parameters/parameters_permit_test.rb @@ -169,7 +169,7 @@ class ParametersPermitTest < ActiveSupport::TestCase test 'arrays are converted at most once' do params = ActionController::Parameters.new(foo: [{}]) - assert params[:foo].equal?(params[:foo]) + assert_same params[:foo], params[:foo] end test "fetch doesnt raise ParameterMissing exception if there is a default" do |