logstash-output-http 5.1.1 → 5.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c81e27750e24a4c4692bf262fb0e5194ef258b2ea2ce60e30054943937266b2f
-  data.tar.gz: b279310c4b9e8d2200cc28963ac6384892c967e5cf8292f15b5a02293df7245a
+  metadata.gz: 0def77507d3ea7b9d83ede431e0853c8f16e77f6249b11b62328e40cd3a0c403
+  data.tar.gz: 85108445896c3ea59793b997e2b73f28dbe7d4fa05cb028af09b3727a8df6e9e
 SHA512:
-  metadata.gz: c4b08091a3c747e633ecedbdfd326b477a973a01a1e530d8afc47bcd470fc9f0f1525265f3dffb098c7ef5c0691968f5983bd1403a2ecfef29ae1f63afe6801f
-  data.tar.gz: 1f264ef6ec26fcb33b8f7ba231bff6810dc9e5e569e292e86f3975de4777ae1868edbde626ad7416409d91163f1d949d5e59c0108707df8e08e96929d7759d11
+  metadata.gz: ee4d0f5fc3152eed88be134bc5c72d99d6f2dccce346fd5bf45bcb3d4ebef72fba9b8b57c2f34dd5053f6b6550fd6a458837edc2bfd0c17d8ff3881f736a4be9
+  data.tar.gz: eaec797c6f1d257c0c4b8ad99d4e8fc329508ec088536f5a204dd169cbeb84efbf2649a753a8684924211be6b51eedd26acef822d8c38dea077dc9399bd634c1

data/CHANGELOG.md

@@ -1,3 +1,6 @@
+## 5.1.2
+  - Add check to avoid hanging pipeline if an empty event array is passed in. #80
+
 ## 5.1.1
   - Update gemspec summary
 

data/LICENSE

@@ -1,4 +1,4 @@
-Copyright (c) 2012–2016 Elasticsearch <http://www.elastic.co>
+Copyright (c) 2012-2018 Elasticsearch <http://www.elastic.co>
 
 Licensed under the Apache License, Version 2.0 (the "License");
 you may not use this file except in compliance with the License.

data/lib/logstash/outputs/http.rb

@@ -113,7 +113,7 @@ class LogStash::Outputs::Http < LogStash::Outputs::Base
   end # def register
 
   def multi_receive(events)
-    send_events(events)
+    send_events(events) unless events.empty?
   end
   
   class RetryTimerTask < java.util.TimerTask
@@ -136,7 +136,7 @@ class LogStash::Outputs::Http < LogStash::Outputs::Base
     
     pending = Queue.new
     events.each {|e| pending << [e, 0]}
-    
+
     while popped = pending.pop
       break if popped == :done
       

data/logstash-output-http.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name            = 'logstash-output-http'
-  s.version         = '5.1.1'
+  s.version         = '5.1.2'
   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"

data/spec/outputs/http_spec.rb

@@ -128,6 +128,12 @@ describe LogStash::Outputs::Http do
       allow(subject).to receive(:log_failure).with(any_args)
     end
 
+    context 'sending no events' do
+      it 'should not block the pipeline' do
+        subject.multi_receive([])
+      end
+    end
+
     context "performing a get" do
       describe "invoking the request" do
         before do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: logstash-output-http
 version: !ruby/object:Gem::Version
-  version: 5.1.1
+  version: 5.1.2
 platform: ruby
 authors:
 - Elastic
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2017-11-07 00:00:00.000000000 Z
+date: 2018-01-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
@@ -133,7 +133,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project:
-rubygems_version: 2.6.11
+rubygems_version: 2.6.13
 signing_key:
 specification_version: 4
 summary: Sends events to a generic HTTP or HTTPS endpoint