Filter out non string header values

castle · Aug 12, 2024 · 423339b · 423339b
1 parent 6d1ccac
commit 423339b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/lib/castle/headers/filter.rb b/lib/castle/headers/filter.rb
@@ -29,6 +29,7 @@ def call
           .keys
           .each_with_object({}) do |header_name, acc|
             next unless header_name.match(VALUABLE_HEADERS)
+            next unless @request_env[header_name].is_a?(String)
 
             formatted_name = @header_format.call(header_name)
             acc[formatted_name] = @request_env[header_name]

diff --git a/spec/lib/castle/headers/filter_spec.rb b/spec/lib/castle/headers/filter_spec.rb
@@ -15,7 +15,8 @@
         'HTTP_USER_AGENT' => 'Mozilla 1234',
         'TEST' => '1',
         'REMOTE_ADDR' => '1.2.3.4',
-        'HTTP_CONTENT_LENGTH' => '0'
+        'HTTP_CONTENT_LENGTH' => '0',
+        'http_accept_language.parser' => -> { 'noop' }
       )
     result[:HTTP_OK] = 'OK'
     result