rollbar 2.16.0 → 2.16.2

data/gemfiles/rails30.gemfile

@@ -55,4 +55,7 @@ end
 
 gem 'resque'
 
+gem 'aws-sdk-sqs'
+gem 'shoryuken'
+
 gemspec :path => '../'

data/gemfiles/rails31.gemfile

@@ -52,4 +52,7 @@ end
 
 gem 'resque'
 
+gem 'aws-sdk-sqs'
+gem 'shoryuken'
+
 gemspec :path => '../'

data/gemfiles/rails32.gemfile

@@ -54,4 +54,7 @@ end
 
 gem 'resque'
 
+gem 'aws-sdk-sqs'
+gem 'shoryuken'
+
 gemspec :path => '../'

data/gemfiles/rails40.gemfile

@@ -54,4 +54,7 @@ else
   gem 'webmock', :require => false
 end
 
-gemspec :path => '../'
+gem 'aws-sdk-sqs'
+gem 'shoryuken'
+
+gemspec :path => '../'

data/gemfiles/rails41.gemfile

@@ -51,4 +51,7 @@ end
 
 gem 'resque'
 
+gem 'aws-sdk-sqs'
+gem 'shoryuken'
+
 gemspec :path => '../'

data/gemfiles/rails42.gemfile

@@ -53,4 +53,7 @@ else
   gem 'webmock', :require => false
 end
 
+gem 'aws-sdk-sqs'
+gem 'shoryuken'
+
 gemspec :path => '../'

data/gemfiles/rails50.gemfile

@@ -55,4 +55,7 @@ end
 
 gem 'webmock', :require => false
 
+gem 'aws-sdk-sqs'
+gem 'shoryuken'
+
 gemspec :path => '../'

data/gemfiles/rails51.gemfile

@@ -55,4 +55,7 @@ end
 
 gem 'webmock', :require => false
 
+gem 'aws-sdk-sqs'
+gem 'shoryuken'
+
 gemspec :path => '../'

data/lib/rollbar/request_data_extractor.rb

@@ -103,6 +103,8 @@ module Rollbar
           {}
         elsif sensitive_headers_list.include?(name)
           { name => Rollbar::Scrubbers.scrub_value(env[header]) }
+        elsif name == 'X-Forwarded-For' && !Rollbar.configuration.collect_user_ip
+          {}
         else
           { name => env[header] }
         end

data/lib/rollbar/tasks/rollbar.cap

@@ -14,58 +14,25 @@ namespace :rollbar do
         :local_username => fetch(:rollbar_user),
         :access_token   => fetch(:rollbar_token),
         :environment    => fetch(:rollbar_env),
-        :revision       => fetch(:rollbar_revision) }
+        :revision       => fetch(:rollbar_revision),
+        :comment        => fetch(:rollbar_comment),
+      }
 
       debug "Building Rollbar POST to #{uri} with #{params.inspect}"
 
       request      = Net::HTTP::Post.new(uri.request_uri)
       request.body = ::JSON.dump(params)
 
-      Net::HTTP.start(uri.host, uri.port, :use_ssl => true) do |http|
+      Net::HTTP.start(uri.host, uri.port, :ENV, :use_ssl => true) do |http|
         http.request(request)
       end
 
       info 'Rollbar notification complete.'
     end
   end
-
-  desc 'Upload sourcemaps to Rollbar.'
-  task :sourcemap do
-    on primary fetch(:rollbar_role) do
-      info "Uploading source maps"
-      warn("You need to upgrade capistrano to '>= 3.1' version in order to correctly upload sourcemaps to Rollbar. (On 3.0, the reported revision will be incorrect.)") if Capistrano::VERSION =~ /^3\.0/
-      url_base = fetch(:rollbar_sourcemaps_minified_url_base)
-      unless url_base
-        warn "No #{rollbar_sourcemaps_minified_url_base} was specified. Sourcemaps won't be uploaded."
-        return
-      end
-      url_base = "http://#{url_base}" unless url_base.index(/https?:\/\//)
-      within release_path do
-        within 'public' do
-          source_maps = capture(:find, '-L', '.', '-name', "'*.js.map'").split("\n")
-          source_maps = source_maps.map { |file| file.gsub(/^\.\//, '') }
-          source_maps.each do |source_map|
-            minified_url = File.join(url_base, source_map)
-            args = *%W(--silent https://api.rollbar.com/api/1/sourcemap -F access_token=#{fetch(:rollbar_token)} -F version=#{fetch(:rollbar_revision)} -F minified_url=#{minified_url} -F source_map=@./#{source_map})
-            info "curl #{args.join(' ')}" # log the command, since capture doesn't output anything
-            api_response_body = capture(:curl, args)
-            begin
-              api_response_json = JSON.parse(api_response_body)
-              if api_response_json["err"] != 0
-                warn "Error uploading sourcemaps: #{api_response_json["message"] || 'Unknown Error'}"
-              end
-            rescue JSON::ParserError => e
-              warn "Error parsing response: #{e.message}. Response body: #{api_response_body}"
-            end
-          end
-        end
-      end
-    end
-  end
 end
 
 namespace :deploy do
-  after 'deploy:compile_assets', 'rollbar:sourcemap'
   after 'deploy:finished', 'rollbar:deploy'
 end
 

data/lib/rollbar/version.rb

@@ -1,3 +1,3 @@
 module Rollbar
-  VERSION = '2.16.0'
+  VERSION = '2.16.2'
 end

data/spec/rollbar/request_data_extractor_spec.rb

@@ -149,6 +149,12 @@ describe Rollbar::RequestDataExtractor do
   
             expect(result[:user_ip]).to be_nil
           end
+          
+          it 'does not extract user\'s IP on X-Forwarded-For header' do
+            result = subject.extract_request_data_from_rack(env)
+
+            expect(result[:headers]['X-Forwarded-For']).to be_nil
+          end
         end
         
         context 'with anonymize_user_ip configuration option enabled' do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rollbar
 version: !ruby/object:Gem::Version
-  version: 2.16.0
+  version: 2.16.2
 platform: ruby
 authors:
 - Rollbar, Inc.
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-05-17 00:00:00.000000000 Z
+date: 2018-06-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: multi_json
@@ -284,7 +284,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.5.1
+rubygems_version: 2.6.10
 signing_key: 
 specification_version: 4
 summary: Reports exceptions to Rollbar