logstash-filter-kv 4.1.0 → 4.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: db1a1524254aafe6af64400a781c90fe02a0e82bcc037d8ce43b652de30dea93
-  data.tar.gz: 36443b889dfde58914512466fe3eadb703fca7c30b665a3d02106f5c6b91cada
+  metadata.gz: 5c2b2e468b13494eda3ab5a4252ec1bc3b1ac73516c7de0f767d5f3a21d1c467
+  data.tar.gz: 145252f085544e2edbceef88a6bef4cc720f6ce745e022150c25657588c3f6d8
 SHA512:
-  metadata.gz: 072403211a806a928e240df180ab2180257b22579db3dd9de9d3ec4065f0e9b75374db3ade99c7e09ad0fc5b66f09701026ee97f81ec3a76c98826b50bd33e94
-  data.tar.gz: 8c7dd7066d669c4f0fea15942f14fc2c80a9412190e31bb0b36cb7382843b3f8839b843b89c72ba930eb04e697e7e5536b01603c78c0666db2b81a49948a0d3d
+  metadata.gz: 718f14bae4895dd2d9cfc252abaace7ae1f784703ec569fb9143946d21378d638889dca25db6f430bdaa25f7838eedda6c2e08eab76d520385701b7171752bcb
+  data.tar.gz: acee9e93136b19746cac5714cf46d5d45e0780caba1ab935aa42a1f26baaa45a111b5214b04cb7dff6499e11a2f6011053d7a46180f8c9b6d78419d80f5b2e13

data/CHANGELOG.md

@@ -1,3 +1,6 @@
+## 4.1.1
+  - bugfix: correctly handle empty values between value separator and field separator (#58)
+
 ## 4.1.0
   - feature: add option to split fields and values using a regex pattern (#55)
 

data/lib/logstash/filters/kv.rb

@@ -356,7 +356,7 @@ class LogStash::Filters::KV < LogStash::Filters::Base
       end
 
       # an unquoted value is a _captured_ sequence of characters or escaped spaces before a `field_split` or EOF.
-      value_patterns << /((?:\\ |.)+?)(?=#{Regexp::union(field_split, eof)})/
+      value_patterns << /((?:\\ |.)*?)(?=#{Regexp::union(field_split, eof)})/
 
       Regexp.union(*value_patterns)
     end

data/logstash-filter-kv.gemspec

@@ -1,7 +1,7 @@
 Gem::Specification.new do |s|
 
   s.name            = 'logstash-filter-kv'
-  s.version         = '4.1.0'
+  s.version         = '4.1.1'
   s.licenses        = ['Apache License (2.0)']
   s.summary         = "Parses key-value pairs"
   s.description     = "This gem is a Logstash plugin required to be installed on top of the Logstash core pipeline using $LS_HOME/bin/logstash-plugin install gemname. This gem is not a stand-alone program"

data/spec/filters/kv_spec.rb

@@ -897,4 +897,37 @@ describe "multi character splitting" do
     end
   end
 
+  describe "handles empty values" do
+    let(:message) { 'a=1|b=|c=3' }
+
+    shared_examples "parse empty values" do
+      it "splits correctly upon empty value" do
+        subject.filter(event)
+
+        expect(event.get("a")).to eq("1")
+        expect(event.get("b")).to be_nil
+        expect(event.get("c")).to eq("3")
+      end
+    end
+
+    context "using char class splitters" do
+      let(:options) {
+        {
+            "field_split" => "|",
+            "value_split" => "=",
+        }
+      }
+      it_behaves_like "parse empty values"
+    end
+
+    context "using pattern splitters" do
+      let(:options) {
+        {
+            "field_split_pattern" => '\|',
+            "value_split_pattern" => "=",
+        }
+      }
+      it_behaves_like "parse empty values"
+    end
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: logstash-filter-kv
 version: !ruby/object:Gem::Version
-  version: 4.1.0
+  version: 4.1.1
 platform: ruby
 authors:
 - Elastic
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2018-02-13 00:00:00.000000000 Z
+date: 2018-03-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement