simple_geoip 0.0.2 → 0.0.3

data/lib/simple_geoip.rb

@@ -7,6 +7,7 @@ module SimpleGeoIP
 
   # parse this the web page and find lat and long, FUCK YEAH!
   def self.find_geo(ip_address)
+    return ["", ""] unless valid_ip? ip_address
     url = lookup_url ip_address
     doc = Nokogiri::HTML open(url)
     table = doc.search('table').last
@@ -23,5 +24,9 @@ module SimpleGeoIP
   def self.lookup_url(ip_address)
     API + ip_address
   end
+  
+  def self.valid_ip?(ip_address)
+    !!ip_address.match(/^(\d{1,3}\.){3}\d{1,3}$/)
+  end
 
 end

data/simple_geoip.gemspec

@@ -8,7 +8,7 @@ Gem::Specification.new do |s|
   ## If your rubyforge_project name is different, then edit it and comment out
   ## the sub! line in the Rakefile
   s.name              = 'simple_geoip'
-  s.version           = '0.0.2'
+  s.version           = '0.0.3'
   s.date              = '2011-05-30'
 
   ## Make sure your summary is short. The description may be as long

data/test/test_basics.rb

@@ -15,5 +15,13 @@ class TestSimpleGeoIP < Test::Unit::TestCase
     lat, long = SimpleGeoIP::find_geo @test_ip
     assert_equal @geo_from_test_ip, [lat, long]
   end
+  
+  def test_valid_ip_address
+    assert SimpleGeoIP::valid_ip?(@test_ip)
+  end
+  
+  def test_invalid_ip_address
+    assert !SimpleGeoIP::valid_ip?("127.0.1/1")
+  end
 
 end

metadata

@@ -2,7 +2,7 @@
 name: simple_geoip
 version: !ruby/object:Gem::Version 
   prerelease: 
-  version: 0.0.2
+  version: 0.0.3
 platform: ruby
 authors: 
 - Chris Ledet