diff options
author | Yves Senn <yves.senn@gmail.com> | 2014-06-06 12:39:23 +0200 |
---|---|---|
committer | Yves Senn <yves.senn@gmail.com> | 2014-06-06 12:39:23 +0200 |
commit | 8d77436f374acf06b46d1e0615b364ebd8acae8d (patch) | |
tree | 52ed7a2fa2611050509c9ed1813e95534695c035 /actionpack | |
parent | e3cd80b4cfda38c9f82342af80dd674236a5d738 (diff) | |
parent | e7b0947afdc8c9f4119d06628070667b9e7875d1 (diff) | |
download | rails-8d77436f374acf06b46d1e0615b364ebd8acae8d.tar.gz rails-8d77436f374acf06b46d1e0615b364ebd8acae8d.tar.bz2 rails-8d77436f374acf06b46d1e0615b364ebd8acae8d.zip |
Merge pull request #15540 from nbibler/http-headers-set
Use a frozen Set instance for CGI_VARIABLES.
Diffstat (limited to 'actionpack')
-rw-r--r-- | actionpack/lib/action_dispatch/http/headers.rb | 29 |
1 files changed, 21 insertions, 8 deletions
diff --git a/actionpack/lib/action_dispatch/http/headers.rb b/actionpack/lib/action_dispatch/http/headers.rb index 3e607bbde1..bc5410dc38 100644 --- a/actionpack/lib/action_dispatch/http/headers.rb +++ b/actionpack/lib/action_dispatch/http/headers.rb @@ -6,14 +6,27 @@ module ActionDispatch # headers = ActionDispatch::Http::Headers.new(env) # headers["Content-Type"] # => "text/plain" class Headers - CGI_VARIABLES = %w( - CONTENT_TYPE CONTENT_LENGTH - HTTPS AUTH_TYPE GATEWAY_INTERFACE - PATH_INFO PATH_TRANSLATED QUERY_STRING - REMOTE_ADDR REMOTE_HOST REMOTE_IDENT REMOTE_USER - REQUEST_METHOD SCRIPT_NAME - SERVER_NAME SERVER_PORT SERVER_PROTOCOL SERVER_SOFTWARE - ) + CGI_VARIABLES = Set.new(%W[ + AUTH_TYPE + CONTENT_LENGTH + CONTENT_TYPE + GATEWAY_INTERFACE + HTTPS + PATH_INFO + PATH_TRANSLATED + QUERY_STRING + REMOTE_ADDR + REMOTE_HOST + REMOTE_IDENT + REMOTE_USER + REQUEST_METHOD + SCRIPT_NAME + SERVER_NAME + SERVER_PORT + SERVER_PROTOCOL + SERVER_SOFTWARE + ]).freeze + HTTP_HEADER = /\A[A-Za-z0-9-]+\z/ include Enumerable |