stop using `@env` in the GET / POST methods

I want to implement this with something besides `@env` in the future, so
lets stop directly referencing it.

1 files changed, 7 insertions, 2 deletions

diff --git a/actionpack/lib/action_dispatch/http/request.rb b/actionpack/lib/action_dispatch/http/request.rb
index 4748a54550..b9fc703fe2 100644
--- a/actionpack/lib/action_dispatch/http/request.rb
+++ b/actionpack/lib/action_dispatch/http/request.rb
@@ -323,7 +323,9 @@ module ActionDispatch
 
     # Override Rack's GET method to support indifferent access
     def GET
-      @env["action_dispatch.request.query_parameters"] ||= normalize_encode_params(super || {})
+      get_header("action_dispatch.request.query_parameters") do |k|
+        set_header k, normalize_encode_params(super || {})
+      end
     rescue Rack::Utils::ParameterTypeError, Rack::Utils::InvalidParameterError => e
       raise ActionController::BadRequest.new(:query, e)
     end
@@ -331,7 +333,9 @@ module ActionDispatch
 
     # Override Rack's POST method to support indifferent access
     def POST
-      @env["action_dispatch.request.request_parameters"] ||= normalize_encode_params(super || {})
+      get_header("action_dispatch.request.request_parameters") do |k|
+        self.request_parameters = normalize_encode_params(super || {})
+      end
     rescue Rack::Utils::ParameterTypeError, Rack::Utils::InvalidParameterError => e
       raise ActionController::BadRequest.new(:request, e)
     end
@@ -352,6 +356,7 @@ module ActionDispatch
     end
 
     def request_parameters=(params)
+      raise if params.nil?
       set_header("action_dispatch.request.request_parameters".freeze, params)
     end