logster 0.9.9 → 1.0.0.0.pre

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 1b6667903d7ea941ef28292bc3e3344e399e1b35
-  data.tar.gz: 024bdf71269ded5bfe341455d5451d0357631796
+  metadata.gz: 27b6e067d72b692b910aa5e441272c5a8fbb9ed5
+  data.tar.gz: f73900bd0bfb40d949d3a80f7e9b28e17239814a
 SHA512:
-  metadata.gz: 08fae884b5a23abd1291f42e4d05e1ceecdbab9a5d0ab5049c3a4ca88ee18488497f3be6fe7f26bae672f66b1fa1eb94d35dd065107105c08f1305e379c03aab
-  data.tar.gz: e0224e2901ff506330d5fe547ae8c2206ceccc52e1d9a15ee1c593368cfc36fc7b7ac6892747bb83dc9a938fa9b2c6e3d03d107202f03f7cb604649c89b8e53c
+  metadata.gz: 1e72c4ea30fcd9a93b461b9363f6158e4ed62fa176038c416f7a4c1d8f6ccb27b991d0eb953a6b1a4525b9df5f60c153bbbbb0b7726e88f94e8e5e7f71d045b8
+  data.tar.gz: abe80ae6300a6623e541f2378834856b96d832239f3ec63924c073331781e9952077e233ec2444bf958051861006352224973101ae982d398be5bf8c64b44009

data/lib/logster/message.rb

@@ -102,7 +102,7 @@ module Logster
 
     # todo - memoize?
     def solved_keys
-      if  (versions=env["application_version"]) &&
+      if (versions=env["application_version"]) &&
           (backtrace && backtrace.length > 0)
         versions = [versions] if String === versions
 

data/lib/logster/redis_store.rb

@@ -191,7 +191,7 @@ module Logster
 
         @redis.hmget(hash_key, message_keys).each do |json|
           message =  Message.from_json(json)
-          unless (ignores & message.solved_keys).empty?
+          unless (ignores & (message.solved_keys || [])).empty?
             delete message
           end
         end

data/lib/logster/version.rb

@@ -1,3 +1,3 @@
 module Logster
-  VERSION = "0.9.9"
+  VERSION = "1.0.0.0.pre"
 end

data/test/logster/test_redis_store.rb

@@ -285,6 +285,16 @@ class TestRedisStore < Minitest::Test
     assert_equal(0, @store.solved.length)
   end
 
+  def test_solving_with_some_missing_version
+
+    m=@store.report(Logger::WARN, "application", "test error1", backtrace: "backtrace1", env: { "application_version" => "xyz"})
+    @store.report(Logger::WARN, "application", "test error1", backtrace: "backtrace1")
+
+    @store.solve(m.key)
+
+    assert_equal(1, @store.latest.count)
+  end
+
   def test_env
     env = Rack::MockRequest.env_for("/test").merge({
       "HTTP_HOST" => "www.site.com",

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: logster
 version: !ruby/object:Gem::Version
-  version: 0.9.9
+  version: 1.0.0.0.pre
 platform: ruby
 authors:
 - UI for viewing logs in Rack
@@ -179,9 +179,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ">"
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
 rubyforge_project: 
 rubygems_version: 2.4.5