logstash-filter-json 0.1.5 → 0.1.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b4abf730a01ac9da979c89a47d2e1b92d657a87f
-  data.tar.gz: 03a580d2eac4b824f0f1ec9f5f79f02e0be9a6f4
+  metadata.gz: 570c551e0acf8c620bac6b1a0ec970c28f17fd19
+  data.tar.gz: 0741d212e884d19071563fdf906cc3b1bce4ee2e
 SHA512:
-  metadata.gz: eb74108d4ad143faded3a10c92892ab429bc56b8521b12a519357c9a993a9139ad518b405ed98beebfc3f4ed57d0b439b9f3e52fdc690264f4d7c31322ee53a5
-  data.tar.gz: 474d3e47d19b1356b9db9da4e68f188e9ac052a3ebee1aaff04cd87ff1e80720fce9e84411f62237cf12bfa95070e285867d7e86966436d9b394c61a0769abf2
+  metadata.gz: 9289ef875a3f2909c9f591c85c33e6325c63f3e3428682a5d4a413861c02cde0d54b56cc8a4dd5323b972f81753e71a8f04bb6703db0223c52c8f654953b17ad
+  data.tar.gz: 1d330052f8cecbe8cbb45d4509f08ec42ba3249c4f604089efaf01fb13442fbc34eefbfa5584b4fcc3d9e39798741255698f344690806588111ab6915452fd7d

data/lib/logstash/filters/json.rb

@@ -90,7 +90,9 @@ class LogStash::Filters::Json < LogStash::Filters::Base
 
       filter_matched(event)
     rescue => e
-      event.tag("_jsonparsefailure")
+      tag = "_jsonparsefailure"
+      event["tags"] ||= []
+      event["tags"] << tag unless event["tags"].include?(tag)
       @logger.warn("Trouble parsing json", :source => @source,
                    :raw => event[@source], :exception => e)
       return

data/logstash-filter-json.gemspec

@@ -1,7 +1,7 @@
 Gem::Specification.new do |s|
 
   s.name            = 'logstash-filter-json'
-  s.version         = '0.1.5'
+  s.version         = '0.1.6'
   s.licenses        = ['Apache License (2.0)']
   s.summary         = "This is a JSON parsing filter. It takes an existing field which contains JSON and expands it into an actual data structure within the Logstash event."
   s.description     = "This gem is a logstash plugin required to be installed on top of the Logstash core pipeline using $LS_HOME/bin/plugin install gemname. This gem is not a stand-alone program"

data/spec/filters/json_spec.rb

@@ -86,4 +86,44 @@ describe LogStash::Filters::Json do
     end
   end
 
+  context "when json could not be parsed" do
+
+    subject(:filter) {  LogStash::Filters::Json.new(config)  }
+
+    let(:message)    { "random_message" }
+    let(:config)     { {"source" => "message"} }
+    let(:event)      { LogStash::Event.new("message" => message) }
+
+    before(:each) do
+      filter.register
+      filter.filter(event)
+    end
+
+    it "add the failure tag" do
+      expect(event).to include "tags"
+    end
+
+    it "uses an array to store the tags" do
+      expect(event['tags']).to be_a Array
+    end
+
+    it "add a json parser failure tag" do
+      expect(event['tags']).to include "_jsonparsefailure"
+    end
+
+    context "there are two different errors added" do
+
+      let(:event)  { LogStash::Event.new("message" => message, "tags" => ["_anotherkinfoffailure"] ) }
+
+      it "pile the different error messages" do
+        expect(event['tags']).to include "_jsonparsefailure"
+      end
+
+      it "keep the former error messages on the list" do
+        expect(event['tags']).to include "_anotherkinfoffailure"
+      end
+    end
+
+  end
+
 end

metadata

@@ -1,17 +1,18 @@
 --- !ruby/object:Gem::Specification
 name: logstash-filter-json
 version: !ruby/object:Gem::Version
-  version: 0.1.5
+  version: 0.1.6
 platform: ruby
 authors:
 - Elastic
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2015-04-20 00:00:00.000000000 Z
+date: 2015-04-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  requirement: !ruby/object:Gem::Requirement
+  name: logstash-core
+  version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
@@ -19,10 +20,7 @@ dependencies:
     - - <
       - !ruby/object:Gem::Version
         version: 2.0.0
-  name: logstash-core
-  prerelease: false
-  type: :runtime
-  version_requirements: !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
@@ -30,20 +28,22 @@ dependencies:
     - - <
       - !ruby/object:Gem::Version
         version: 2.0.0
+  prerelease: false
+  type: :runtime
 - !ruby/object:Gem::Dependency
-  requirement: !ruby/object:Gem::Requirement
+  name: logstash-devutils
+  version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
-  name: logstash-devutils
-  prerelease: false
-  type: :development
-  version_requirements: !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+  prerelease: false
+  type: :development
 description: This gem is a logstash plugin required to be installed on top of the Logstash core pipeline using $LS_HOME/bin/plugin install gemname. This gem is not a stand-alone program
 email: info@elastic.co
 executables: []