logster 2.9.0 → 2.9.5

data/lib/logster/message.rb

@@ -99,7 +99,7 @@ module Logster
           env["time"] = @timestamp || get_timestamp
         end
       end
-      @env = Message.populate_from_env(env)
+      self.env = Message.populate_from_env(env)
     end
 
     def self.default_env
@@ -113,7 +113,8 @@ module Logster
 
     # in its own method so it can be overridden
     def grouping_hash
-      { message: self.message, severity: self.severity, backtrace: self.backtrace }
+      message = self.message.gsub(/[0-9a-f]+/i, "X")
+      { message: message, severity: self.severity, backtrace: self.backtrace }
     end
 
     # todo - memoize?

data/lib/logster/middleware/debug_exceptions.rb

@@ -14,13 +14,11 @@ class Logster::Middleware::DebugExceptions < ActionDispatch::DebugExceptions
     exception = wrapper.exception
 
     Logster.config.current_context.call(env) do
-      location = exception.backtrace[0]
-
       Logster.logger.add_with_opts(
         ::Logger::Severity::FATAL,
-        "#{exception.class} (#{exception})\n#{location}",
+        "#{exception.class} (#{exception})\n#{wrapper.application_trace.join("\n")}",
         "web-exception",
-        backtrace: exception.backtrace.join("\n"),
+        backtrace: wrapper.full_trace.join("\n"),
         env: env
       )
     end

data/lib/logster/redis_store.rb

@@ -342,7 +342,10 @@ module Logster
     def rate_limited?(ip_address, perform: false, limit: 60)
       key = ip_rate_limit_key(ip_address)
 
-      limited = @redis.exists(key)
+      limited = @redis.call([:exists, key])
+      if Integer === limited
+        limited = limited != 0
+      end
 
       if perform && !limited
         @redis.setex key, limit, ""

data/lib/logster/version.rb

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

data/logster.gemspec

@@ -33,5 +33,5 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "guard-minitest"
   spec.add_development_dependency "timecop"
   spec.add_development_dependency "byebug", "~> 11.1.0"
-  spec.add_development_dependency "rubocop-discourse", "~> 2.1.2"
+  spec.add_development_dependency "rubocop-discourse"
 end

data/test/logster/middleware/test_reporter.rb

@@ -28,19 +28,19 @@ class TestReporter < Minitest::Test
 
     reporter = Logster::Middleware::Reporter.new(nil)
     env = Rack::MockRequest.env_for("/logs/report_js_error?message=hello")
-    status, = reporter.call(env)
+    reporter.call(env)
 
     assert_equal(Logger::Severity::WARN, Logster.store.latest[-1].severity)
 
     reporter = Logster::Middleware::Reporter.new(nil)
     env = Rack::MockRequest.env_for("/logs/report_js_error?message=hello&severity=invalid")
-    status, = reporter.call(env)
+    reporter.call(env)
 
     assert_equal(Logger::Severity::WARN, Logster.store.latest[-1].severity)
 
     reporter = Logster::Middleware::Reporter.new(nil)
     env = Rack::MockRequest.env_for("/logs/report_js_error?message=hello&severity=error")
-    status, = reporter.call(env)
+    reporter.call(env)
 
     assert_equal(Logger::Severity::ERROR, Logster.store.latest[-1].severity)
   end

data/test/logster/test_base_store.rb

@@ -188,4 +188,16 @@ class TestBaseStore < Minitest::Test
       refute_includes(store.reported.first.env.keys.map(&:to_sym), :backtrace)
     end
   end
+
+  def test_envs_with_invalid_encoding_dont_raise_errors
+    msg = @store.report(
+      Logger::WARN,
+      '',
+      'me have invalid encoding',
+      env: {
+        axe: "a\xF1xasa"
+      }
+    )
+    assert_equal("a�xasa", msg.env[:axe])
+  end
 end

data/test/logster/test_redis_store.rb

@@ -1044,6 +1044,24 @@ class TestRedisStore < Minitest::Test
     end
   end
 
+  def test_messages_that_differ_only_by_numbers_or_hashes_are_merged
+    config_reset(allow_grouping: true) do
+      first_message = <<~TEXT
+        DistributedMutex("download_20450e291e8f1e5ba03ca7f20fb7d9da570c94a6"):
+        held for too long, expected max: 60 secs, took an extra 73 secs
+      TEXT
+      msg = @store.report(Logger::WARN, '', first_message, backtrace: caller)
+
+      second_message = <<~TEXT
+        DistributedMutex("download_e09ae082c60a351dedec67ed869652862b232a0b"):
+        held for too long, expected max: 60 secs, took an extra 287 secs
+      TEXT
+      msg2 = @store.report(Logger::WARN, '', second_message, backtrace: caller)
+
+      assert_equal(msg.key, msg2.key)
+    end
+  end
+
   private
 
   def config_reset(configs)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: logster
 version: !ruby/object:Gem::Version
-  version: 2.9.0
+  version: 2.9.5
 platform: ruby
 authors:
 - Sam Saffron
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-06-11 00:00:00.000000000 Z
+date: 2021-01-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -126,16 +126,16 @@ dependencies:
   name: rubocop-discourse
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 2.1.2
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 2.1.2
+        version: '0'
 description: UI for viewing logs in Rack
 email:
 - sam.saffron@gmail.com
@@ -143,9 +143,9 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/ci.yml"
 - ".gitignore"
 - ".rubocop.yml"
-- ".travis.yml"
 - CHANGELOG.md
 - Gemfile
 - Guardfile
@@ -295,7 +295,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: UI for viewing logs in Rack

data/.travis.yml

@@ -1,17 +0,0 @@
-language: ruby
-
-matrix:
-  fast_finish: true
-
-rvm:
-  - 2.5.3
-  - 2.6.5
-  - 2.7.0
-
-services:
-  - redis-server
-
-before_install:
-  - gem install bundler -v 1.17.2
-
-sudo: false