email_checker 0.0.1 → 0.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 287b145d67ecd07f435e662d23557d5383365e9e
-  data.tar.gz: 91062933ded8f2ff2e4081cf148cde5eac0bf137
+  metadata.gz: e6187abcb5b21a81b97d88b4f77879d237abb301
+  data.tar.gz: 3288b04c61442116a410264f2f36634c6b47f87c
 SHA512:
-  metadata.gz: 5eacfdcdb7e892fec853b94e8d880161a379c76e806785b2530693825473360c6a72742440432479ae0dc9aef17569c1a2bbe2e40432b68af783f7ae12b791a2
-  data.tar.gz: df4ae4f2a51a082e59451dd631060bf5d37fcb2367b93337ec9ba66c81b668b33f0a1a280f890a822cabaecb55b5a4f29f333738cde212d677270cfc484f6887
+  metadata.gz: db83aa157497c79588ad6763b7fe7589593a4ad58dd0e87578bf678cef9ac9c34ab4f678f8f09756e3ac0d8b712b25b21daa68ab7c65dd3a143495bf540dbf2b
+  data.tar.gz: aa6251f1b167ebb547bc43523d256b448e406d157657b422a3b82b83c1bfe0aa5a3aee2d9f39b52942af1b09d74e12ed7f92cd4adabbd1a31145c211d3a6d927

data/.gitignore

@@ -13,3 +13,4 @@
 *.a
 mkmf.log
 *.gem
+rubocop.html

data/Rakefile

@@ -3,4 +3,3 @@ require 'bundler/gem_tasks'
 task :console do
   exec 'irb -r email_checker -I ./lib'
 end
-

data/email_checker.gemspec

@@ -8,8 +8,8 @@ Gem::Specification.new do |spec|
   spec.version       = EmailChecker::VERSION
   spec.authors       = ['Genaro Madrid']
   spec.email         = ['genmadrid@gmail.com']
-  spec.summary       = %q{Check if an email address is can receive E-mails.}
-  spec.description   = %q{Validates, at some degree, that the email you want to send to it's valid and exists.}
+  spec.summary       = 'Check if an email address is can receive E-mails.'
+  spec.description   = "Validates, at some degree, that the email you want to send to it's valid and exists."
   spec.homepage      = 'https://github.com/genaromadrid/email_checker'
   spec.license       = 'MIT'
 

data/lib/email_checker/checker.rb

@@ -23,20 +23,17 @@ module EmailChecker
 
       def smtp
         return @smtp if @smtp
-        tries ||= 5
         domain.mx_servers.each do |server|
-          @smtp = Net::SMTP.start(server[:address], 25, EmailChecker.config.verifier_domain)
-          return @smtp if @smtp
+          @smtp = connect(server[:address])
+          break if @smtp
         end
-        fail EmailChecker::ServerConnectionError, "Unable to connect to any of the mail servers for #{@email}"
-      rescue EmailChecker::ServerConnectionError => e
-        fail EmailChecker::ServerConnectionError, e.message
+        fail EmailChecker::ServerConnectionError, "Unable to connect to any of the mail servers for #{@email}" unless @smtp
+      end
+
+      def connect(address)
+        Net::SMTP.start(address, 25, EmailChecker.config.verifier_domain)
       rescue => e
-        if (tries -= 1) > 0
-          retry
-        else
-          fail EmailChecker::FailureError, e.message
-        end
+        raise EmailChecker::FailureError, e.message
       end
 
       def mailfrom
@@ -49,7 +46,7 @@ module EmailChecker
         if e.message[/^550/]
           return false
         else
-          fail EmailChecker::FailureError, e.message
+          raise EmailChecker::FailureError, e.message
         end
       end
 

data/lib/email_checker/config.rb

@@ -8,9 +8,7 @@ module EmailChecker
       def reset
         @verifier_email = nil
         @test_mode = false
-        if defined?(Rails) && defined?(Rails.env) && Rails.env.test?
-          @test_mode = true
-        end
+        @test_mode = true if defined?(Rails) && defined?(Rails.env) && Rails.env.test?
       end
 
       def verifier_email=(verifier_email)
@@ -19,6 +17,6 @@ module EmailChecker
       end
     end
 
-    self.reset
+    reset
   end
 end

data/lib/email_checker/domain.rb

@@ -1,6 +1,5 @@
 module EmailChecker
   class Domain
-
     def initialize(domain)
       @domain = domain
     end
@@ -10,7 +9,7 @@ module EmailChecker
     # @return [type] [description]
     def valid?
       return false unless @domain
-      Timeout::timeout(SERVER_TIMEOUT) do
+      Timeout.timeout(SERVER_TIMEOUT) do
         return true if valid_mx_records?
         return true if a_records?
       end
@@ -42,7 +41,7 @@ module EmailChecker
     end
 
     def mx_records
-      @mx_records ||= dns.getresources(@domain, Resolv::DNS::Resource::IN::MX).sort_by {|mx| mx.preference}
+      @mx_records ||= dns.getresources(@domain, Resolv::DNS::Resource::IN::MX).sort_by(&:preference)
     end
 
     def mx_servers

data/lib/email_checker/version.rb

@@ -1,3 +1,3 @@
 module EmailChecker
-  VERSION = '0.0.1'
+  VERSION = '0.0.2'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: email_checker
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
 platform: ruby
 authors:
 - Genaro Madrid
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-12-16 00:00:00.000000000 Z
+date: 2015-12-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler