logstash-filter-http 1.5.0 → 1.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1eab107931f9800b2f79a338dcc58a5f6bf0f18703738de822ba663c1d3e882c
-  data.tar.gz: b59faea346ae0c7414b539d91d8d053254667cf01c700b1169b3eb6145d65768
+  metadata.gz: 81c8d1ec8c914081f8ce16a71a22be25e114afa967fb327c133503a5cca7263f
+  data.tar.gz: 37f64bd9d01e2361881fccf2d638b6471372fe1b69acd81e36efdfccb2e61916
 SHA512:
-  metadata.gz: e760ced473f745bdf0d50cabf3803a2abbb1ddd2e9203fa20021899636c289bad1cbb09c6b56b0b1626dae01af2530f62200c343836f6ef447de39e621694cb5
-  data.tar.gz: 4796a68ece2ffe8f977c7d985325eb30485457cac6e6d11e59dccf90c49fa6d107f9b3f3c5dddcb992b36297f13ed16c36405c953537f229c16b98f186acd6b6
+  metadata.gz: 42b529894dd5d0b34b398ed1522b6b1472ca0d934d8d52e932017b7f9dc9d6412d1ff17265bad21ba8bdd6ed0ea29d6a04627a476ebcdd375be1a2676a959d2f
+  data.tar.gz: b0111d7b61fe6f092526e919302e01ec73ab1e6c1c75d2329e8337ad67a14036195509e1357320f8228340cff0e85bba0b92da440f1ace5e6e52e49eba5baf3c

data/CHANGELOG.md

@@ -1,3 +1,6 @@
+## 1.5.1
+  - Don't process response when the body is empty. [#50](https://github.com/logstash-plugins/logstash-filter-http/pull/50)
+
 ## 1.5.0
   - Added standardized SSL settings and deprecates their non-standard counterparts. Deprecated settings will continue to work, and will provide pipeline maintainers with guidance toward using their standardized counterparts [#49](https://github.com/logstash-plugins/logstash-filter-http/pull/49)
   - Added new `ssl_truststore_path`, `ssl_truststore_password`, and `ssl_truststore_type` settings for configuring SSL-trust using a PKCS-12 or JKS trust store, deprecating their `truststore`, `truststore_password`, and `truststore_type` counterparts.

data/lib/logstash/filters/http.rb

@@ -136,6 +136,8 @@ EOF
     event.set(@target_headers, headers)
     return if @verb == 'head' # Since HEAD requests will not contain body, we need to set only header
 
+    return if body.nil? #Return on empty bodys (e.g. 204 response)
+
     if headers_has_json_content_type?(headers)
       begin
         parsed = LogStash::Json.load(body)

data/logstash-filter-http.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name = 'logstash-filter-http'
-  s.version = '1.5.0'
+  s.version = '1.5.1'
   s.licenses = ['Apache License (2.0)']
   s.summary = 'This filter requests data from a RESTful Web Service.'
   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 logstash-filter-http. This gem is not a stand-alone program'

data/spec/filters/http_spec.rb

@@ -61,6 +61,14 @@ describe LogStash::Filters::Http do
         end
       end
 
+      context "when there is no body" do
+        let(:response) { [204, {}, nil] }
+
+        it "doesn't write a body to the event" do
+          expect(event.get('[gw-response]')).to be_nil
+        end
+      end
+
       context 'with body target' do
 
         let(:config) { super().merge "target_body" => '[rest]' }

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: logstash-filter-http
 version: !ruby/object:Gem::Version
-  version: 1.5.0
+  version: 1.5.1
 platform: ruby
 authors:
 - Elastic
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-09-01 00:00:00.000000000 Z
+date: 2024-01-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement