aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack
diff options
context:
space:
mode:
authorYves Senn <yves.senn@gmail.com>2014-06-06 12:39:23 +0200
committerYves Senn <yves.senn@gmail.com>2014-06-06 12:39:23 +0200
commit8d77436f374acf06b46d1e0615b364ebd8acae8d (patch)
tree52ed7a2fa2611050509c9ed1813e95534695c035 /actionpack
parente3cd80b4cfda38c9f82342af80dd674236a5d738 (diff)
parente7b0947afdc8c9f4119d06628070667b9e7875d1 (diff)
downloadrails-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.rb29
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