Short circuit empty sources, update DB

Fixes #134

CHANGELOG.md

@@ -1,3 +1,7 @@
+## 5.0.3
+ - Skip lookup operation if source field contains an empty string 
+ - Update of the GeoIP2 DB
+
 ## 5.0.2
   - Update gemspec summary
 

logstash-filter-geoip.gemspec

@@ -1,7 +1,7 @@
 Gem::Specification.new do |s|
 
   s.name            = 'logstash-filter-geoip'
-  s.version         = '5.0.2'
+  s.version         = '5.0.3'
   s.licenses        = ['Apache License (2.0)']
   s.summary         = "Adds geographical information about an IP address"
   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"

spec/filters/geoip_spec.rb

@@ -93,8 +93,8 @@
 
       it "should set other subfields of 'target' properly" do
         expect(event.get("target").to_hash.keys.sort).to eq(["city_name", "ip", "region_name"])
-        expect(event.get("[target][city_name]")).to eq("Salem")
-        expect(event.get("[target][region_name]")).to eq("New Hampshire")
+        expect(event.get("[target][city_name]")).to eq("Watertown")
+        expect(event.get("[target][region_name]")).to eq("Massachusetts")
       end
 
     end
@@ -183,6 +183,14 @@
       sample("ip" => "~") do
         expect{ subject }.to_not raise_error
       end
+
+      sample("ip" => "") do
+        expect{ subject }.to_not raise_error
+      end
+
+      sample("ip" => "      ") do
+        expect{ subject }.to_not raise_error
+      end
     end
 
     describe "filter method outcomes" do

src/main/java/org/logstash/filters/GeoIPFilter.java

@@ -114,17 +114,21 @@ public boolean handleEvent(RubyEvent rubyEvent) {
     if (input == null) {
       return false;
     }
-
     String ip;
 
     if (input instanceof List) {
       ip = (String) ((List) input).get(0);
+
     } else if (input instanceof String) {
       ip = (String) input;
     } else {
       throw new IllegalArgumentException("Expected input field value to be String or List type");
     }
 
+    if (ip.trim().isEmpty()){
+      return false;
+    }
+
     Map<String, Object> geoData = new HashMap<>();
 
     try {

vendor.json

@@ -1,7 +1,7 @@
 [
     {
         "url": "http://geolite.maxmind.com/download/geoip/database/GeoLite2-City.mmdb.gz",
-        "sha1": "fe165dab8ed0f2e980fbce81d05f5e120b0b1fa9"
+        "sha1": "faccb3c92fd5bee0261e6e7640a79c7e37624d16"
     },
     {
         "url": "https://s3.amazonaws.com/download.elasticsearch.org/logstash/maxmind/GeoLite2-ASN.mmdb",