timber 2.0.21 → 2.0.22

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: f65e44d5582af9d177dabd02f2229adda5ea094b
-  data.tar.gz: a113bb27776a6403b3b237b49d33c7fbac962758
+  metadata.gz: fb727abf7308983fedd10e2b44e661ce337a5fed
+  data.tar.gz: 4dd342bfd405154dc7e2cc670789745c8879f01b
 SHA512:
-  metadata.gz: 10b371f99e2c4f4203526917a770140095581e17a12b72fba44dfc8db8b2151a198b5e771527fb715f6c9c4fe9e0d1c56772ff6663fb31c07f76bd95210339d4
-  data.tar.gz: e0b9fd2b91d7de66b4bee5305337afeb5017ec42fff2c76e8374c957af5d88445701aa029fca846f2f7e7778e4ffa7231989ea6c05d3814f307e244e5b2607d0
+  metadata.gz: 1e781929108c46a6a4a6979ed4255bb57e2bb8ee86397f95b8da4a517e795778dd715347d993ab627eefe951a565d9890b91d2729c73717c8a8224ce125f2c2c
+  data.tar.gz: ac45f8c6b93f67b1829b0771e74b37968ea22e2f413e9bf4ac7952e0c2b292536ebd18f82b807a9721708ba5f20b15be50e85ca7710fa0d96d76287f87d4a7cf

data/lib/timber/util/request.rb

@@ -6,19 +6,26 @@ end
 if defined?(::Rack::Request)
   module Timber
     module Util
+      # @private
       class Request < ::Rack::Request
+        HTTP_PREFIX = 'HTTP_'.freeze
+
         def body_content
           content = body.read
           body.rewind
           content
         end
 
+        # Returns a list of request headers. The rack env contains a lot of data, this function
+        # identifies those that were the actual request headers.
         def headers
-          @headers ||= ::Hash[*@env.select {|k,v| k.start_with? 'HTTP_'}
-            .collect {|k,v| [k.sub(/^HTTP_/, ''), v]}
-            .collect {|k,v| [k.split('_').collect(&:capitalize).join('-'), v]}
-            .sort
-            .flatten]
+          @headers ||= ::Hash[
+            *@env.select { |k,v| k.is_a?(String) && k.start_with?(HTTP_PREFIX) }
+              .collect { |k,v| [k.sub(/^#{HTTP_PREFIX}/, ''), v] }
+              .collect { |k,v| [k.split('_').collect(&:capitalize).join('-'), v] }
+              .sort
+              .flatten
+          ]
         end
 
         def ip

data/lib/timber/version.rb

@@ -1,3 +1,3 @@
 module Timber
-  VERSION = "2.0.21"
+  VERSION = "2.0.22"
 end

data/spec/timber/util/request_spec.rb

@@ -0,0 +1,10 @@
+require "spec_helper"
+
+describe Timber::Util::Request, :rails_23 => true do
+  describe ".headers" do
+    it "should ignore symbol keys" do
+      req = described_class.new({test: "value"})
+      expect(req.headers).to eq({})
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: timber
 version: !ruby/object:Gem::Version
-  version: 2.0.21
+  version: 2.0.22
 platform: ruby
 authors:
 - Timber Technologies, Inc.
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-05-22 00:00:00.000000000 Z
+date: 2017-05-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: msgpack
@@ -243,6 +243,7 @@ files:
 - spec/timber/log_entry_spec.rb
 - spec/timber/logger_spec.rb
 - spec/timber/util/http_event_spec.rb
+- spec/timber/util/request_spec.rb
 - timber.gemspec
 homepage: https://github.com/timberio/timber-ruby
 licenses: []
@@ -300,3 +301,4 @@ test_files:
 - spec/timber/log_entry_spec.rb
 - spec/timber/logger_spec.rb
 - spec/timber/util/http_event_spec.rb
+- spec/timber/util/request_spec.rb