logstash-filter-dns 3.0.9 → 3.0.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3c33d4c587fb242c61000133efcf81920b77f9bb153adb9b23b92af9764442c8
-  data.tar.gz: e70b5f973930d77e4b140b79a72d86512bbf5f59b0aba07b5722dfa2755b242a
+  metadata.gz: 91e354fcb8db6795eda49ca3c4bcf0971eed81af89d90fec4a1f6b4ec3a9bade
+  data.tar.gz: 377b1a63e87937e249fc439d5d332dbdd5b9fffca84830c53dd7175c9ba9b3db
 SHA512:
-  metadata.gz: 10cba307e1989269ee3b203772380198bbfc0a1a81e2c038a491de9e92890e49b3403b3496092aea6bdbf8e54909e111b58d2883075b61894a1b705d29ed0cca
-  data.tar.gz: 3b8f9360f6625f3d76dc664b5e43121fa3372c2fb15755717c60cea80c559ce52c2de26898823eac2d359aee534362c9f36eb0042412563338bf412bd9e31978
+  metadata.gz: e5ad88694cf8f01c733a798d1bef59616eaf63d196caea8c49bd3e45a843e401d84c93ebf0125a59e621d7b3a6bd1227109e0e8ed660cd41db3a88a415f92ff9
+  data.tar.gz: 628ec7747dafe97e40f54241f1b75f6a333aeeb0ee2841c0939143a911e45102c3068c7b6f6df9f6f2c42b777860a3db412471780b9d78f8f2bfd2dc0a826ca3

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 3.0.10
+  - Log timeouts as warn instead of error #43
+  - Allow concurrent queries when cache enabled #42
+
 ## 3.0.9
   - Logging improvement to include DNS resolution failure reason #36
 

data/lib/logstash/filters/dns.rb

@@ -142,7 +142,12 @@ class LogStash::Filters::DNS < LogStash::Filters::Base
       begin
         return if @failed_cache && @failed_cache[raw] # recently failed resolv, skip
         if @hit_cache
-          address = @hit_cache.getset(raw) { retriable_getaddress(raw) }
+          address = @hit_cache[raw]
+          if address.nil?
+            if address = retriable_getaddress(raw)
+              @hit_cache[raw] = address
+            end
+          end
         else
           address = retriable_getaddress(raw)
         end
@@ -153,7 +158,7 @@ class LogStash::Filters::DNS < LogStash::Filters::Base
         return
       rescue Resolv::ResolvTimeout, Timeout::Error
         @failed_cache[raw] = true if @failed_cache
-        @logger.error("DNS: timeout on resolving the hostname.",
+        @logger.warn("DNS: timeout on resolving the hostname.",
                       :field => field, :value => raw)
         return
       rescue SocketError => e
@@ -209,7 +214,12 @@ class LogStash::Filters::DNS < LogStash::Filters::Base
       begin
         return if @failed_cache && @failed_cache.key?(raw) # recently failed resolv, skip
         if @hit_cache
-          hostname = @hit_cache.getset(raw) { retriable_getname(raw) }
+          hostname = @hit_cache[raw]
+          if hostname.nil?
+            if hostname = retriable_getname(raw)
+              @hit_cache[raw] = hostname
+            end
+          end
         else
           hostname = retriable_getname(raw)
         end
@@ -220,7 +230,7 @@ class LogStash::Filters::DNS < LogStash::Filters::Base
         return
       rescue Resolv::ResolvTimeout, Timeout::Error
         @failed_cache[raw] = true if @failed_cache
-        @logger.error("DNS: timeout on resolving address.",
+        @logger.warn("DNS: timeout on resolving address.",
                       :field => field, :value => raw)
         return
       rescue SocketError => e

data/logstash-filter-dns.gemspec

@@ -1,7 +1,7 @@
 Gem::Specification.new do |s|
 
   s.name            = 'logstash-filter-dns'
-  s.version         = '3.0.9'
+  s.version         = '3.0.10'
   s.licenses        = ['Apache License (2.0)']
   s.summary         = "Performs a standard or reverse DNS lookup"
   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"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: logstash-filter-dns
 version: !ruby/object:Gem::Version
-  version: 3.0.9
+  version: 3.0.10
 platform: ruby
 authors:
 - Elastic
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2018-04-04 00:00:00.000000000 Z
+date: 2018-05-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement