logstash-filter-http 1.2.1 → 1.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4d5469fab82031f6b39e205f18142291d9896c97cd4b650c5124ff0f661ca945
-  data.tar.gz: 878169ea102e2c342fabed654cc8d265df31a7a935fbe8dd4260f79c2b393fce
+  metadata.gz: cf9e6e5719322e393443523446a54d9940048cd2dc540d4f71afedf7dd58b1dd
+  data.tar.gz: 72869368a417ccc28e522ab0ef47a3910d3bdb8565a9c215108a3cc3d8f088ca
 SHA512:
-  metadata.gz: 6f0f5d51f50ec667de851cf7223d004062d3946c1bc0ee4b849edb3de26bc3da5246a168502c3a169023d646772835f79ebf990daf615085e4ce57a8ed5cc2db
-  data.tar.gz: 25d1125226d4c5adbc0bb74bac0ba1d0e7aa02eba55d765e0a15c084638efea664bab10e618eeb80d85e36cc4e94140708c93b26fab6843e3c9a15d13947732a
+  metadata.gz: 2c76225e97cc973becbace99e5f2af1cbbebcdc73d264e735dc6064b5d7d1ecd88a51f0665bed3df2c6571d209c61ffa044be69f009762454713d367f96e653b
+  data.tar.gz: 2cffbf09fc61fd9a0fc7594e7f828497e9c8e098e056053c8791a2d9bb16533cedf2b31f3cf934f1bd5cc40e9d2574f007895a90ff469d962fb4a6d6f19eec52

data/CHANGELOG.md

@@ -1,3 +1,12 @@
+## 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)
+ 
+## 1.3.0
+  - Feat: support ssl_verification_mode option [#37](https://github.com/logstash-plugins/logstash-filter-http/pull/37)
+
 ## 1.2.1
   - Fix: do not set content-type if provided by user [#36](https://github.com/logstash-plugins/logstash-filter-http/pull/36)
 

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.2.1'
+  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'
@@ -28,7 +28,7 @@ Gem::Specification.new do |s|
   # Gem dependencies
   s.add_runtime_dependency 'logstash-core-plugin-api', '>= 1.60', '<= 2.99'
   s.add_runtime_dependency 'logstash-mixin-ecs_compatibility_support', '~>1.2'
-  s.add_runtime_dependency 'logstash-mixin-http_client', '>= 5.0.0', '< 9.0.0'
+  s.add_runtime_dependency "logstash-mixin-http_client", ">= 7.2.0", '< 9.0.0'
   s.add_runtime_dependency 'logstash-mixin-validator_support', '~> 1.0'
 
   s.add_development_dependency 'logstash-devutils'

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.2.1
+  version: 1.4.1
 platform: ruby
 authors:
 - Elastic
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-12-29 00:00:00.000000000 Z
+date: 2022-05-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
@@ -49,7 +49,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 5.0.0
+        version: 7.2.0
     - - "<"
       - !ruby/object:Gem::Version
         version: 9.0.0
@@ -60,7 +60,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 5.0.0
+        version: 7.2.0
     - - "<"
       - !ruby/object:Gem::Version
         version: 9.0.0