logstash-filter-rest 0.2.0 → 0.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 70cf7db34fec1104a8eb742ce5633c11769a66d5
-  data.tar.gz: c265edbd76879148168331adc875befb458ec0a6
+  metadata.gz: 2af5930ffb298925b7a340afd929ac9c85559f4f
+  data.tar.gz: 57e160ff830adbc6b63d22968e4f20047b1779a3
 SHA512:
-  metadata.gz: fac6c51358cd5cd6e9990807f03b169aeca591ea828c11f1b718b39639e99770edd534f896aa0b0ec1da8ed3e441d7807f34f8cfdf26e1403c75d73eeeaddbcd
-  data.tar.gz: 70a3d59b3d30ebc070e715f97be104c4e2161b59405b5ac488963a78d1eb8b400dd7b1d71025c8d382ca52d34d13de56401331793089a37d200c9d85872d5e86
+  metadata.gz: bedafb7870ae6c5d0fc22c81f4fcedd5b1384d6524afd8394da949502f6af8ce9d551615fa73751e112db6e44acfb51b90854874ac4199f0cc3cf53e55a7951a
+  data.tar.gz: cb947ea8b5cbd4f666afcb297f2e45cc2f8bf5655db1f16b66de7a7a649eefc01332fb6e3156076b92ada1986d9e469027c5a8552f1880ab0a69be55f86efc1e

data/lib/logstash/filters/rest.rb

@@ -106,7 +106,7 @@ class LogStash::Filters::Rest < LogStash::Filters::Base
   public
 
   def register
-    @request = normalize_request(@request)
+    @request = normalize_request(@request).freeze
   end # def register
 
   private
@@ -190,11 +190,12 @@ class LogStash::Filters::Rest < LogStash::Filters::Base
 
   def filter(event)
     return unless filter?(event)
-    @request[2][:params] = sprint(@sprintf, @request[2][:params], event) if @request[2].key?(:params)
-    @request[2][:body] = sprint(@sprintf, @request[2][:body], event) if @request[2].key?(:body)
-    @request[1] = sprint(@sprintf, @request[1], event)
+    request = @request.dup
+    request[2][:params] = sprint(@sprintf, @request[2][:params], event) if request[2].key?(:params)
+    request[2][:body] = sprint(@sprintf, @request[2][:body], event) if request[2].key?(:body)
+    request[1] = sprint(@sprintf, @request[1], event)
 
-    code, body = request_http(@request)
+    code, body = request_http(request)
     if code.between?(200, 299)
       event = process_response(body, event)
       @logger.debug? && @logger.debug('Sucess received', :code => code, :body => body)
@@ -202,7 +203,7 @@ class LogStash::Filters::Rest < LogStash::Filters::Base
       @logger.debug? && @logger.debug('Http error received', :code => code, :body => body)
       if @fallback.empty?
         @tag_on_rest_failure.each { |tag| event.tag(tag) }
-        @logger.error('Error in Rest filter', :request => @request, :json => @json, :code => code, :body => body)
+        @logger.error('Error in Rest filter', :request => request, :json => @json, :code => code, :body => body)
       else
         event = add_to_event(@fallback, event)
         @logger.debug? && @logger.debug('Setting fallback', :fallback => @fallback)

data/logstash-filter-rest.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name = 'logstash-filter-rest'
-  s.version = '0.2.0'
+  s.version = '0.2.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'

data/spec/filters/rest_spec.rb

@@ -63,6 +63,12 @@ describe LogStash::Filters::Rest do
       expect(subject['rest']['id']).to eq(10)
       expect(subject['rest']).to_not include("fallback")
     end
+    sample("message" => "9") do
+      expect(subject).to include('rest')
+      expect(subject['rest']).to include("id")
+      expect(subject['rest']['id']).to eq(9)
+      expect(subject['rest']).to_not include("fallback")
+    end
   end
   describe "Set to Rest Filter Get without params http error" do
     let(:config) do <<-CONFIG
@@ -134,6 +140,12 @@ describe LogStash::Filters::Rest do
       expect(subject['rest'][0]['userId']).to eq(10)
       expect(subject['rest']).to_not include("fallback")
     end
+    sample("message" => "9") do
+      expect(subject).to include('rest')
+      expect(subject['rest'][0]).to include("userId")
+      expect(subject['rest'][0]['userId']).to eq(9)
+      expect(subject['rest']).to_not include("fallback")
+    end
   end
   describe "Set to Rest Filter Post with params" do
     let(:config) do <<-CONFIG

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: logstash-filter-rest
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.2.1
 platform: ruby
 authors:
 - Lucas Henning
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2016-08-20 00:00:00.000000000 Z
+date: 2016-08-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
@@ -135,7 +135,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project:
-rubygems_version: 2.6.4
+rubygems_version: 2.4.8
 signing_key:
 specification_version: 4
 summary: This filter requests data from a RESTful Web Service.