logstash-filter-rest 0.5.0 → 0.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 4fac99e78c6f1c75edbbf92e74fede3862dca591
-  data.tar.gz: d7ccae6caf784862a5f998ff48116cf96b44030a
+  metadata.gz: 2468297289c6978316c5d27c664d5c162cd0412c
+  data.tar.gz: 6bb1e1e69ef6ac4409e14e4170b81ea46f657094
 SHA512:
-  metadata.gz: b49a566196685abb817a20f4438f9fedcf14a33bb8c82a9a766ed0bdb91e7cbeeb9f18746e4eba277fbce034ad798c32361bf6e54e7bfd1c4ccfebb2eb891ad7
-  data.tar.gz: 15d9d11fb62f6bda9b4d67e70e753eb5b0e2a447f054213bcc8e9831a06b5358ffc635ea57e5988cc05fd8760bebcecace6bba7a5c9412d773a83aaed666d20f
+  metadata.gz: 01227dc244aeb7c369dd7d5127423ce63e21cedc63797052ec30164c7800906141a2b5249de77c4adb8d6f053d7d949790a5a8120d9e1fdcb42f2e256932ab15
+  data.tar.gz: b5d5599af27a350e07122ff29cea1dd94d9f4866005568c38a062bb1a4ac6a4e576161c457f5c59e8b145248943b46a51c9bd7e738714629e186b9daea2edd82

data/README.md

@@ -12,7 +12,7 @@ This logstash filter provides an easy way to access RESTful Resources within log
 ### 1. Installation
 You can use the built-in plugin tool of Logstash to install the filter:
 ```
-$LS_HOME/bin/plugin install logstash-filter-rest
+$LS_HOME/bin/logstash-plugin install logstash-filter-rest
 ```
 
 Or you can build it yourself:
@@ -20,7 +20,7 @@ Or you can build it yourself:
 git clone https://github.com/lucashenning/logstash-filter-rest.git
 bundle install
 gem build logstash-filter-rest.gemspec
-$LS_HOME/bin/plugin install logstash-filter-rest-0.1.0.gem
+$LS_HOME/bin/logstash-plugin install logstash-filter-rest-0.1.0.gem
 ```
 
 ### 2. Filter Configuration

data/lib/logstash/filters/rest.rb

@@ -272,18 +272,26 @@ class LogStash::Filters::Rest < LogStash::Filters::Base
     @logger.debug? && @logger.debug('Parsed request',
                                     :request => @request)
 
-    code, body = request_http(@request)
-    if code.between?(200, 299)
-      @logger.debug? && @logger.debug('Sucess received',
+    client_error = nil
+    begin
+      code, body = request_http(@request)
+    rescue StandardError => e
+      client_error = e
+    end
+
+    if !client_error && code.between?(200, 299)
+      @logger.debug? && @logger.debug('Success received',
                                       :code => code, :body => body)
       process_response(body, event)
     else
       @logger.debug? && @logger.debug('Http error received',
-                                      :code => code, :body => body)
+                                      :code => code, :body => body,
+                                      :client_error => client_error)
       if @fallback.empty?
         @logger.error('Error in Rest filter',
                       :request => @request, :json => @json,
-                      :code => code, :body => body)
+                      :code => code, :body => body,
+                      :client_error => client_error)
         @tag_on_rest_failure.each { |tag| event.tag(tag) }
       else
         @logger.debug? && @logger.debug('Setting fallback',

data/logstash-filter-rest.gemspec

@@ -1,9 +1,9 @@
 Gem::Specification.new do |s|
   s.name = 'logstash-filter-rest'
-  s.version = '0.5.0'
+  s.version = '0.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/plugin install gemname. This gem is not a stand-alone program'
+  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-rest. This gem is not a stand-alone program'
   s.authors = ['Lucas Henning', 'Gandalf Buscher']
   s.email = 'mail@hurb.de'
   s.homepage = 'https://github.com/lucashenning/logstash-filter-rest/'

data/spec/filters/rest_spec.rb

@@ -328,4 +328,21 @@ describe LogStash::Filters::Rest do
       expect { subject }.to raise_error(LogStash::ConfigurationError)
     end
   end
+  describe 'http client throws exception' do
+    let(:config) do <<-CONFIG
+      filter {
+        rest {
+          request => {
+            url => 'invalid_url'
+          }
+          target => 'rest'
+        }
+      }
+    CONFIG
+    end
+    sample('message' => 'some text') do
+      expect(subject).to_not include('rest')
+      expect(subject.get('tags')).to include('_restfailure')
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: logstash-filter-rest
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.5.1
 platform: ruby
 authors:
 - Lucas Henning
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2016-12-16 00:00:00.000000000 Z
+date: 2017-01-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
@@ -71,7 +71,7 @@ dependencies:
     - - "<"
       - !ruby/object:Gem::Version
         version: 2.0.0
-description: This gem is a logstash plugin required to be installed on top of the Logstash core pipeline using $LS_HOME/bin/plugin install gemname. This gem is not a stand-alone program
+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-rest. This gem is not a stand-alone program
 email: mail@hurb.de
 executables: []
 extensions: []