timber 2.6.0 → 2.6.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c0070dc6695f710539f3827bf7587eefaec03979
-  data.tar.gz: 775ab9aec9ad834e629ccdddd0ff08e045d76378
+  metadata.gz: 695d1eb93a0e891ecc1a3eb47ee4888d0d30fdbc
+  data.tar.gz: d1075432948895089dec419679a8fb12194a7631
 SHA512:
-  metadata.gz: c8dc7a46f2a2d30a778290a9238d64c0725b6be34f62ff98485fd1d2305e69c359e966d5ff3f111ad31168a364ebf9e9c15e5af9103aaa0c81b334fa0bd8e925
-  data.tar.gz: 4fe9551096dde4ad67b8e9876e6c1d6bf4f51b9eb155f633ff178abe3dd1544ff867de64d40d065522106ec5e87f1c0d537956a43ed4daa53ac5c00756c03824
+  metadata.gz: 6318864253dec537d4e604161e5877873407744d57d738519fc24091a72126dbe3429cf4be85d106f09f28ab41d4513218407094d27aa296575429e22b30f8b9
+  data.tar.gz: 90cb2e402a913de26c656bb9b5bef4f49ebeff58e3b06c56b9c335888dd07f434d63f6da473cb993da1f9e537ccead50fabf36877bd7cd6652d98d105fa6a38d

data/CHANGELOG.md

@@ -7,6 +7,12 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 
 ## [Unreleased]
 
+## [2.6.1] - 2017-11-28
+
+### Fixed
+
+  - Fixes an issue where a reference to the current custom context map was being capture during log line creation and then later self-modifying to make the context invalid.
+
 ## [2.6.0] - 2017-11-28
 
 ### Fixed
@@ -112,7 +118,8 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
     instead of applying back pressure.
 
 
-[Unreleased]: https://github.com/timberio/timber-ruby/compare/v2.6.0...HEAD
+[Unreleased]: https://github.com/timberio/timber-ruby/compare/v2.6.1...HEAD
+[2.6.1]: https://github.com/timberio/timber-ruby/compare/v2.6.0...v2.6.1
 [2.6.0]: https://github.com/timberio/timber-ruby/compare/v2.5.1...v2.6.0
 [2.5.1]: https://github.com/timberio/timber-ruby/compare/v2.5.0...v2.5.1
 [2.5.0]: https://github.com/timberio/timber-ruby/compare/v2.4.0...v2.5.0

data/lib/timber/current_context.rb

@@ -120,7 +120,13 @@ module Timber
     # since the context can change as execution proceeds. Note that individual contexts
     # should be immutable, and we implement snapshot caching as a result of this assumption.
     def snapshot
-      @snapshot ||= hash.clone
+      @snapshot ||= begin
+        snapshot = hash.clone
+        if snapshot.key?(:custom)
+          snapshot[:custom] = hash[:custom].clone
+        end
+        snapshot
+      end
     end
 
     private

data/lib/timber/log_devices/http.rb

@@ -147,6 +147,7 @@ module Timber
           req['Authorization'] = authorization_payload
           req['Content-Type'] = CONTENT_TYPE
           req['User-Agent'] = USER_AGENT
+
           req.body = msgs.to_msgpack
           req
         end

data/lib/timber/version.rb

@@ -1,3 +1,3 @@
 module Timber
-  VERSION = "2.6.0"
+  VERSION = "2.6.1"
 end

data/spec/timber/current_context_spec.rb

@@ -98,6 +98,18 @@ describe Timber::CurrentContext, :rails_23 => true do
     end
   end
 
+  describe ".snapshot" do
+    it "shoud properly snapshot custom contexts" do
+      snapshot = nil
+
+      described_class.with({build: {version: "1.0.0"}}) do
+        snapshot = described_class.instance.snapshot
+      end
+
+      expect(snapshot[:custom]).to eq({:build=>{:version=>"1.0.0"}})
+    end
+  end
+
   describe ".with" do
     it "should merge the context and cleanup on block exit" do
       expect(described_class.instance.send(:hash)[:custom]).to be_nil

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: timber
 version: !ruby/object:Gem::Version
-  version: 2.6.0
+  version: 2.6.1
 platform: ruby
 authors:
 - Timber Technologies, Inc.