logstash-output-http 5.7.0 → 5.7.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ea9e1064040e6d7b09ea61a4f77de09bd1a56186d824ec6c26b870c2ac466975
-  data.tar.gz: b1ae025edddfc791309aeb4edd7a0a93a2b961358b73484b83625ad029cfceb7
+  metadata.gz: 4ee250c7e18f8f86d45a9976fdad992c6b329d862c153c1ee786032bb1fca022
+  data.tar.gz: 346c7c1027d694bb03288a271a72bdde0f8f1d37a162dd2b2f6b6e8073e45f37
 SHA512:
-  metadata.gz: 2594c607190fcf16a6c6793a19ee2c14bab9d787f688f567ec1ec0a06669a771d7213e0189e5675ef211e9bc0d313b728a0cc613817bc2af554ea6ad04b4d554
-  data.tar.gz: 74ae9117f75b81e93e706d2acefe4c8e42d15139521aa4cf5310a21a7a9f4756763fbfa242de552cc222ed0550a03501489c55ccc56a16e0c71658c5d8c9e66e
+  metadata.gz: 889fc141a67c6eddc1dc35f87da3fde5fcc56c5f294662a370bcc401a8a7848937ad5e1c912d1234ae68f1de3779bab2648e8588249c6f3e6ef6087dbf5c0a34
+  data.tar.gz: d6ef06b950a4763025625ca3dc9e8c09bd086f50a42d0bce35f0de64ab445cac4a3e54980343374a69ab615fdc1940a7df4b57e9077c3ddcc956213e11476bdb

data/CHANGELOG.md

@@ -1,3 +1,6 @@
+## 5.7.1
+  - Added new development `rackup` dependency to fix tests 
+
 ## 5.7.0
   - Added new `ssl_enabled` setting for enabling/disabling the SSL configurations [#144](https://github.com/logstash-plugins/logstash-output-http/pull/144)
 

data/logstash-output-http.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name            = 'logstash-output-http'
-  s.version         = '5.7.0'
+  s.version         = '5.7.1'
   s.licenses        = ['Apache License (2.0)']
   s.summary         = "Sends events to a generic HTTP or HTTPS endpoint"
   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"
@@ -25,4 +25,7 @@ Gem::Specification.new do |s|
   s.add_development_dependency 'logstash-devutils'
   s.add_development_dependency 'sinatra'
   s.add_development_dependency 'webrick'
+
+  # Pin to avoid using new Fiber-based implementation that breaks tests here
+  s.add_development_dependency 'rackup', "< 2.1.0"
 end

data/spec/outputs/http_spec.rb

@@ -255,7 +255,7 @@ describe LogStash::Outputs::Http do
       end
 
       let(:last_request) { TestApp.last_request }
-      let(:body) { last_request.body.read }
+      let(:body) {  read_last_request_body(last_request) }
       let(:content_type) { last_request.env["CONTENT_TYPE"] }
 
       it "should receive the request" do
@@ -458,7 +458,7 @@ RSpec.describe LogStash::Outputs::Http do # different block as we're starting we
   after  { subject.close }
 
   let(:last_request) { TestApp.last_request }
-  let(:last_request_body) { last_request.body.read }
+  let(:last_request_body) { read_last_request_body(last_request) }
 
   let(:event) { LogStash::Event.new("message" => "hello!") }
 
@@ -530,3 +530,10 @@ RSpec.describe LogStash::Outputs::Http do # different block as we're starting we
   end if tls_version_enabled_by_default?('TLSv1.3') && JOpenSSL::VERSION > '0.12' # due WEBrick uses OpenSSL
 
 end
+
+# Pre-emptively rewind the retrieval of `last_request.body` - for form based endpoints, the body
+# is placed in params, and body is empty, requiring a `rewind` for the body to be available for comparison
+def read_last_request_body(last_request)
+  last_request.body.rewind
+  last_request.body.read
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: logstash-output-http
 version: !ruby/object:Gem::Version
-  version: 5.7.0
+  version: 5.7.1
 platform: ruby
 authors:
 - Elastic
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-06-19 00:00:00.000000000 Z
+date: 2024-11-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
@@ -20,8 +20,8 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '2.99'
   name: logstash-core-plugin-api
-  prerelease: false
   type: :runtime
+  prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -40,8 +40,8 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 8.0.0
   name: logstash-mixin-http_client
-  prerelease: false
   type: :runtime
+  prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -57,8 +57,8 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
   name: logstash-devutils
-  prerelease: false
   type: :development
+  prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -71,8 +71,8 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
   name: sinatra
-  prerelease: false
   type: :development
+  prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -85,13 +85,27 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
   name: webrick
-  prerelease: false
   type: :development
+  prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: 2.1.0
+  name: rackup
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: 2.1.0
 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
@@ -133,7 +147,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.33
+rubygems_version: 3.3.26
 signing_key:
 specification_version: 4
 summary: Sends events to a generic HTTP or HTTPS endpoint