logster 2.7.0 → 2.7.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 106a0763acb86a917c9a1b2f3a3e80cfc4e9ef789a1904e76499d62ff5f66298
-  data.tar.gz: 74aef16f2994ddde18124e9a914decb094fa275813eb31d396190f4bbd4d4e35
+  metadata.gz: 5e2fb360e66b19d26e80b991ce21ffea457434700d7610c10e68e429f94dbf42
+  data.tar.gz: f26c5e4b6d19284fa2e3ed2de99b4abe73c4c1a4c64afdf151c537d3c8dfbd1a
 SHA512:
-  metadata.gz: ecc9724cc453bbf7c909c7ab9f6857250afc246dd1bc535ad7d29bbb1df7917c8f64b7e64f5a78d686f7118386c4359791fde6d8ca9e945e5f8213e3d76d8466
-  data.tar.gz: ae722de77352b4c355ae7f330d3af1697142394ea63f5a5a67dcfaeb1e6fe9594278e4a0deb2f647a0ba6a89f4297ed16592d421b1b98709991e95160a427f8a
+  metadata.gz: 27fb6774bbe4db2d727b760273fe14c005719e1dc8ff1338cce78fbcae9cdd624baf420302903c8cef0e7f073713e72e9002312fe143faa26e3090431152bd99
+  data.tar.gz: cf1474a67c00acc807a297b496ba47a1051d316a5b19ae2c551d30d5ba12a04602124ab58ffc5077b0a4931ab3779973e04cb6536dccfc0ee1de3f115ed621c0

data/CHANGELOG.md

@@ -1,5 +1,9 @@
 # CHANGELOG
 
+- 2020-03-03: 2.7.1
+
+  - FIX: Stop env mutation to allow all chained loggers to have the same env (#110)
+
 - 2020-02-26: 2.7.0
 
   - FEATURE: Make log message max length configurable and bump the default to 2000 (#109)

data/lib/logster/base_store.rb

@@ -156,7 +156,7 @@ module Logster
       msg = truncate_message(msg)
       message = Logster::Message.new(severity, progname, msg, opts[:timestamp], count: opts[:count])
 
-      env = opts[:env] || {}
+      env = opts[:env]&.dup || {}
       backtrace = opts[:backtrace]
       if Hash === env && env[:backtrace]
         # Special - passing backtrace through env

data/lib/logster/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Logster
-  VERSION = "2.7.0"
+  VERSION = "2.7.1"
 end

data/test/logster/test_base_store.rb

@@ -176,4 +176,16 @@ class TestBaseStore < Minitest::Test
   ensure
     Logster.config.maximum_message_length = orig
   end
+
+  def test_chained_loggers_shouldnt_mutate_env_passed_to_them
+    logger = Logster::Logger.new(@store)
+    other_store = Logster::TestStore.new
+    other_logger = Logster::Logger.new(other_store)
+    logger.chain(other_logger)
+    logger.add(Logger::WARN, "this is warning", '', { env: { backtrace: '11' } })
+    [@store, other_store].each do |store|
+      assert_equal('11', store.reported.first.backtrace)
+      refute_includes(store.reported.first.env.keys.map(&:to_sym), :backtrace)
+    end
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: logster
 version: !ruby/object:Gem::Version
-  version: 2.7.0
+  version: 2.7.1
 platform: ruby
 authors:
 - UI for viewing logs in Rack
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-02-26 00:00:00.000000000 Z
+date: 2020-03-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler