rollbar 2.15.1 → 2.15.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 73754e6a1e6bf9d7914571a174c10265cd0d85fa
-  data.tar.gz: 8c1f3715b20d672299b4be4a1fdbd8cc7fd7a602
+  metadata.gz: 437007ddf3cda14f45d74e3bec9c4b16a0249345
+  data.tar.gz: 51d29703e1202335322a5386516e8ae3f85c48b7
 SHA512:
-  metadata.gz: 702e9699419e0877522940dc1b6fe782c8d73dbb118f38b80dfb566e9f3db070fa3f8ccb244313d90c0d3e813c032fbc06d5f4cc9815be1fe47710725fc07b46
-  data.tar.gz: a077c1f297ea59b496ad9f172e8aafc900fe91971f44fd4f42ddb23675e1554352dbf563f99a5b8c2f0fa1e36a5599cbc9a591f36f6fe2400a92ad7ca0e303d0
+  metadata.gz: 1e670cee39adebd4ecef829babd67c3db917ea2060bc3da396af5de471b3c24d59a046e48023922033c209b9def0b5717ff6340adb94cabad288f761cd36f51c
+  data.tar.gz: 2bdaab51831ea7398ad86aed457da70db6f8106599528ce81bfe5aa5cf9a2bfe422c5e451eaf2ac96c1caf032d509f7c6dd9d22d00cebd095753207ebb9b38b7

data/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Change Log
 
+## 2.15.2
+
+- Fix how person data is injected into javascript
+
 ## 2.15.1
 
 - Update rollbar.js to v2.2.3 [#630](https://github.com/rollbar/rollbar-gem/pull/630)

data/README.md

@@ -1,4 +1,4 @@
-# Rollbar [![Build Status](https://api.travis-ci.org/rollbar/rollbar-gem.svg?branch=v2.15.1)](https://travis-ci.org/rollbar/rollbar-gem/branches)
+# Rollbar [![Build Status](https://api.travis-ci.org/rollbar/rollbar-gem.svg?branch=v2.15.2)](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/delay/shoryuken.rb

@@ -8,13 +8,13 @@ module Rollbar
     class Shoryuken
       include ::Shoryuken::Worker
 
-      # not allowing bulk, to not double-report rollbars if one of them failed in bunch.
-      shoryuken_options :queue => queue_name, :auto_delete => true, :body_parser => :json, :retry_intervals => [60, 180, 360, 120_0, 360_0, 186_00]
-
       def self.queue_name
         "rollbar_#{Rollbar.configuration.environment}"
       end
 
+      # not allowing bulk, to not double-report rollbars if one of them failed in bunch.
+      shoryuken_options :queue => queue_name, :auto_delete => true, :body_parser => :json, :retry_intervals => [60, 180, 360, 120_0, 360_0, 186_00]
+
       ## responsible for performing job. - payload is a json parsed body of the message.
       def perform(_sqs_message, payload)
         Rollbar.process_from_async_handler(payload)

data/lib/rollbar/middleware/js.rb

@@ -2,6 +2,7 @@ require 'rack'
 require 'rack/response'
 
 require 'rollbar/request_data_extractor'
+require 'rollbar/util'
 
 module Rollbar
   module Middleware
@@ -107,7 +108,7 @@ module Rollbar
       end
 
       def config_js_tag(env)
-        js_config = config[:options].dup
+        js_config = Rollbar::Util.deep_copy(config[:options])
 
         add_person_data(js_config, env)
 

data/lib/rollbar/tasks/rollbar.cap

@@ -36,7 +36,7 @@ end
 
 namespace :load do
   task :defaults do
-    set :rollbar_user,      Proc.new { ENV['USER'] || ENV['USERNAME'] }
+    set :rollbar_user,      Proc.new { fetch :local_user, ENV['USER'] || ENV['USERNAME'] }
     set :rollbar_env,       Proc.new { fetch :rails_env, 'production' }
     set :rollbar_token,     Proc.new { abort "Please specify the Rollbar access token, set :rollbar_token, 'your token'" }
     set :rollbar_role,      Proc.new { :app }

data/lib/rollbar/version.rb

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

data/spec/rollbar/middleware/js_spec.rb

@@ -253,6 +253,12 @@ END
         let(:headers) do
           { 'Content-Type' => content_type }
         end
+        let(:config) do
+          {
+            :enabled => true,
+            :options => { :foo => :bar, :payload => { :a => 42 } }
+          }
+        end
         let(:env) do
           {
             'rollbar.person_data' => {
@@ -266,6 +272,7 @@ END
           {
             :foo => :bar,
             :payload => {
+              :a => 42,
               :person => {
                 :id => 100,
                 :username => 'foo',
@@ -295,6 +302,20 @@ END
 
             expect(new_body).not_to include('person')
           end
+
+          it 'doesnt include old data when called a second time' do
+            _, _, _ = subject.call({
+                'rollbar.person_data' => {
+                  :id => 100,
+                  :username => 'foo',
+                  :email => 'foo@bar.com'
+                }
+            })
+            _, _, response = subject.call(env)
+            new_body = response.body.join
+
+            expect(new_body).not_to include('person')
+          end
         end
       end
     end

metadata

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