airbrake-ruby 2.2.5 → 2.2.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 2c6bcda11cc4df213ee240b89afd624859204780
-  data.tar.gz: 50f609e5e47c2129750b43f7f98bebf13794ef52
+  metadata.gz: 468e348697a972a35ec849945767df8b9f41f46a
+  data.tar.gz: 3e419d4cf9bb00c0ecfb64aa5bac605d2d8ecb82
 SHA512:
-  metadata.gz: daa0b464c5ca8818a006e0a10cec1ff185ffbf2b3602b0b57f1fda4b69032e17d3dbb512054b5a89a15ef0571d698526570c4a8c538c2d7ee1120b1b58158b9c
-  data.tar.gz: 845cd20d7caf62ab809db418cc38178e4dbf257e3436181250ffc14af851087be444315cd0af26830f0b3223e09df8d7a99aeaacc386d13b408766d62f621768
+  metadata.gz: 0debe52953f21f4af23a04895eaaffb9f48a1eb5aa04a32ec58a26a542799cb3411e15f2f8e54d0514f58b861eba889f3d4e013ff28405c1b2144c73a036d815
+  data.tar.gz: c552c5522d8d53e66f17322da98650ffb3e155dca191161fbeae2efecce9052092e1ca9808f7798f30593f7750984d9f6d2ec179d89ed8b47318f2ce1a003e6a

data/lib/airbrake-ruby/filters/thread_filter.rb

@@ -21,6 +21,12 @@ module Airbrake
         Numeric
       ].freeze
 
+      ##
+      # Variables starting with this prefix are not attached to a notice.
+      # @see https://github.com/airbrake/airbrake-ruby/issues/229
+      # @return [String]
+      IGNORE_PREFIX = '_'.freeze
+
       def initialize
         @weight = 110
       end
@@ -51,12 +57,14 @@ module Airbrake
 
       def thread_variables(th)
         th.thread_variables.map.with_object({}) do |var, h|
+          next if var.to_s.start_with?(IGNORE_PREFIX)
           h[var] = sanitize_value(th.thread_variable_get(var))
         end
       end
 
       def fiber_variables(th)
         th.keys.map.with_object({}) do |key, h|
+          next if key.to_s.start_with?(IGNORE_PREFIX)
           h[key] = sanitize_value(th[key])
         end
       end

data/lib/airbrake-ruby/version.rb

@@ -4,5 +4,5 @@
 module Airbrake
   ##
   # @return [String] the library version
-  AIRBRAKE_RUBY_VERSION = '2.2.5'.freeze
+  AIRBRAKE_RUBY_VERSION = '2.2.6'.freeze
 end

data/spec/filters/thread_filter_spec.rb

@@ -119,6 +119,18 @@ RSpec.describe Airbrake::Filters::ThreadFilter do
         )
       end
     end
+
+    it "ignores thread variables starting with an underscore" do
+      var = :__recursive_key__
+
+      new_thread do |th|
+        th.thread_variable_set(var, :bingo)
+        subject.call(notice)
+      end
+
+      thread_variables = notice[:params][:thread][:thread_variables]
+      expect(thread_variables).to be_nil
+    end
   end
 
   describe "fiber variables" do
@@ -254,4 +266,16 @@ RSpec.describe Airbrake::Filters::ThreadFilter do
     subject.call(notice)
     expect(notice[:params][:thread][:safe_level]).to eq(0)
   end
+
+  it "ignores fiber variables starting with an underscore" do
+    key = :__recursive_key__
+
+    new_thread do |th|
+      th[key] = :bingo
+      subject.call(notice)
+    end
+
+    fiber_variables = notice[:params][:thread][:fiber_variables]
+    expect(fiber_variables[key]).to be_nil
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: airbrake-ruby
 version: !ruby/object:Gem::Version
-  version: 2.2.5
+  version: 2.2.6
 platform: ruby
 authors:
 - Airbrake Technologies, Inc.
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-05-23 00:00:00.000000000 Z
+date: 2017-06-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec