add error message when domain parsing errors out

nikhgupta · Mar 28, 2020 · 6dcdfb0 · 6dcdfb0
1 parent a74d938
commit 6dcdfb0
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/lib/mail_provider.rb b/lib/mail_provider.rb
@@ -4,7 +4,10 @@ module MailProvider
   ROOT_DIR = File.dirname(File.dirname(__FILE__))
 
   class Error < StandardError; end
-  class TrieLoadError < StandardError; end
+  class ClientError < Error; end
+  class ServerError < Error; end
+  class TrieLoadError < ServerError; end
+  class DomainParsingError < ClientError; end
 end
 
 require 'faraday'

diff --git a/lib/mail_provider/checker.rb b/lib/mail_provider/checker.rb
@@ -34,6 +34,8 @@ def check(str, summarize: false)
 
       add_score_to_response
       @response
+    rescue PublicSuffix::Error => e
+      raise DomainParsingError, e.message
     end
 
     def find(str)