logstash-filter-http 1.4.0 → 1.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9cc14c2d1874c8df2c795535e90ee6b7e85070d601039ab0a8f5c91da874af16
-  data.tar.gz: 21c3aa078560f4900fb8db35eca35abb60b71798a224164f2a41cd4682e2f755
+  metadata.gz: cf9e6e5719322e393443523446a54d9940048cd2dc540d4f71afedf7dd58b1dd
+  data.tar.gz: 72869368a417ccc28e522ab0ef47a3910d3bdb8565a9c215108a3cc3d8f088ca
 SHA512:
-  metadata.gz: aba589b66860997ba29e3edaa5564c04d897ed9502fc46b263a356ae80a7c768aa1570873ced5865aa7c66523e1aa04054ef604d61e68e4c3644f548456d82a1
-  data.tar.gz: 8974fa26e8127e6a7a880d44c8df7be1ac4ec4435dc38a6912f589f1d1f7bc4eae2154f8cd2fdb3307a5a67896df2fd95e692460d182f65f5fb69cc12f2b58e1
+  metadata.gz: 2c76225e97cc973becbace99e5f2af1cbbebcdc73d264e735dc6064b5d7d1ecd88a51f0665bed3df2c6571d209c61ffa044be69f009762454713d367f96e653b
+  data.tar.gz: 2cffbf09fc61fd9a0fc7594e7f828497e9c8e098e056053c8791a2d9bb16533cedf2b31f3cf934f1bd5cc40e9d2574f007895a90ff469d962fb4a6d6f19eec52

data/CHANGELOG.md

@@ -1,3 +1,6 @@
+## 1.4.1
+  - Fix: don't process response body for HEAD requests [#40](https://github.com/logstash-plugins/logstash-filter-http/pull/40)
+
 ## 1.4.0
   - Feat: added ssl_supported_protocols option [#38](https://github.com/logstash-plugins/logstash-filter-http/pull/38)
  

data/lib/logstash/filters/http.rb

@@ -134,6 +134,7 @@ EOF
   def process_response(body, headers, event)
     content_type, _ = headers.fetch("content-type", "").split(";")
     event.set(@target_headers, headers)
+    return if @verb == 'head' # Since HEAD requests will not contain body, we need to set only header
     if content_type == "application/json"
       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.4.0'
+  s.version = '1.4.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

@@ -191,7 +191,6 @@ describe LogStash::Filters::Http do
     before(:each) { subject.register }
     let(:response) { [200, {}, "Bom dia"] }
     let(:url) { "http://stringsize.com" }
-    let(:config) { super().merge "body" => body }
 
     describe "format" do
       let(:config) { super().merge "body_format" => body_format, "body" => body }
@@ -277,6 +276,16 @@ describe LogStash::Filters::Http do
         subject.filter(event)
       end
     end
+    context "when the verb is HEAD" do
+      let(:config) { super().merge("verb" => "HEAD") }
+      before(:each) do
+        allow(subject).to receive(:request_http).and_return(response)
+      end
+      it "does not include the body" do
+        subject.filter(event)
+        expect(event).to_not include("body")
+      end
+    end
   end
   describe "verb" do
     let(:response) { [200, {}, "Bom dia"] }

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: logstash-filter-http
 version: !ruby/object:Gem::Version
-  version: 1.4.0
+  version: 1.4.1
 platform: ruby
 authors:
 - Elastic
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-03-28 00:00:00.000000000 Z
+date: 2022-05-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement