rollbar 2.15.2 → 2.15.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 437007ddf3cda14f45d74e3bec9c4b16a0249345
-  data.tar.gz: 51d29703e1202335322a5386516e8ae3f85c48b7
+  metadata.gz: 576f76f0fa343a7b3482542fb2f8ea5fd8607322
+  data.tar.gz: f808fbbbe3f19cf5d66482c083b6470b363788cd
 SHA512:
-  metadata.gz: 1e670cee39adebd4ecef829babd67c3db917ea2060bc3da396af5de471b3c24d59a046e48023922033c209b9def0b5717ff6340adb94cabad288f761cd36f51c
-  data.tar.gz: 2bdaab51831ea7398ad86aed457da70db6f8106599528ce81bfe5aa5cf9a2bfe422c5e451eaf2ac96c1caf032d509f7c6dd9d22d00cebd095753207ebb9b38b7
+  metadata.gz: fb71b77242924373d75631fce0a40f316490a79b67cb7f635c6929a0b0c9e53382d61b9753c4267d25e8aeb9661fb6857bcf5a6f4594707df51e06130f9fa5e9
+  data.tar.gz: 9ec91943bfc0c18b9744b64561fcb94979ae4153366d881c3596d28627365a017bb04505f0c360967a7cc0b6dc0da3b731f5f8cfce7cbe29fdbf4225bae549d2

data/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Change Log
 
+## 2.15.3
+
+- Fix a bug when host is nil when we are trying to extract data about a request
+  [#637](https://github.com/rollbar/rollbar-gem/pull/637).
+- Make a copy of extra data passed in so we don't modify frozen objects
+  [#638](https://github.com/rollbar/rollbar-gem/pull/638)
+
 ## 2.15.2
 
 - Fix how person data is injected into javascript

data/README.md

@@ -1,4 +1,4 @@
-# Rollbar [![Build Status](https://api.travis-ci.org/rollbar/rollbar-gem.svg?branch=v2.15.2)](https://travis-ci.org/rollbar/rollbar-gem/branches)
+# Rollbar [![Build Status](https://api.travis-ci.org/rollbar/rollbar-gem.svg?branch=v2.15.3)](https://travis-ci.org/rollbar/rollbar-gem/branches)
 
 <!-- RemoveNext -->
 [Rollbar](https://rollbar.com) is an error tracking service for Ruby and other languages. The Rollbar service will alert you of problems with your code and help you understand them in a ways never possible before. We love it and we hope you will too.

data/lib/rollbar/notifier.rb

@@ -337,7 +337,7 @@ module Rollbar
         end
       end
 
-      [message, exception, extra]
+      [message, exception, Rollbar::Util.deep_dup(extra)]
     end
 
     def lookup_exception_level(orig_level, exception, use_exception_level_filters)

data/lib/rollbar/request_data_extractor.rb

@@ -112,8 +112,8 @@ module Rollbar
       forwarded_proto = env['HTTP_X_FORWARDED_PROTO'] || env['rack.url_scheme'] || ''
       scheme = forwarded_proto.split(',').first
 
-      forwarded_host = env['HTTP_X_FORWARDED_HOST'] || env['HTTP_HOST'] || env['SERVER_NAME']
-      host = forwarded_host && forwarded_host.split(',').first.strip
+      host = env['HTTP_X_FORWARDED_HOST'] || env['HTTP_HOST'] || env['SERVER_NAME'] || ''
+      host = host.split(',').first.strip unless host.empty?
 
       path = env['ORIGINAL_FULLPATH'] || env['REQUEST_URI']
       unless path.nil? || path.empty?
@@ -121,8 +121,8 @@ module Rollbar
       end
 
       port = env['HTTP_X_FORWARDED_PORT']
-      if port && !(scheme.downcase == 'http' && port.to_i == 80) && \
-         !(scheme.downcase == 'https' && port.to_i == 443) && \
+      if port && !(!scheme.nil? && scheme.downcase == 'http' && port.to_i == 80) && \
+         !(!scheme.nil? && scheme.downcase == 'https' && port.to_i == 443) && \
          !(host.include? ':')
         host = host + ':' + port
       end

data/lib/rollbar/util.rb

@@ -62,6 +62,21 @@ module Rollbar
       end
     end
 
+    def self.deep_dup(obj)
+      if obj.is_a?(::Hash)
+        result = obj.dup
+        obj.each { |k, v| result[k] = deep_dup(v)}
+        result
+      elsif obj.is_a?(Array)
+        result = obj.dup
+        result.clear
+        obj.each { |v| result << deep_dup(v)}
+        result
+      else
+        obj.dup
+      end
+    end
+
     def self.deep_merge(hash1, hash2)
       hash1 ||= {}
       hash2 ||= {}

data/lib/rollbar/version.rb

@@ -1,3 +1,3 @@
 module Rollbar
-  VERSION = '2.15.2'
+  VERSION = '2.15.3'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rollbar
 version: !ruby/object:Gem::Version
-  version: 2.15.2
+  version: 2.15.3
 platform: ruby
 authors:
 - Rollbar, Inc.
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-09-13 00:00:00.000000000 Z
+date: 2017-09-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: multi_json