truemail 0.1.6 → 0.1.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d2c535ef7ee89b62217b36df966ba3d94422498a0431d87218540d4d12073ad9
-  data.tar.gz: 462b13314a2d223d7db3b305d108a8a8f052197fc5cbd311001dfdfea2b6bddc
+  metadata.gz: f597eb293f10835d12b766da633b9d1808424760924a89472416729e60241c23
+  data.tar.gz: a4fe4b473856a44d312bfb15fdd831a412d5764fedc33274f5923b75b59b5ce0
 SHA512:
-  metadata.gz: b2dc0174969794428067aba261022536cbca33895686b89f88be6ee83d4c5c97caa1e8ea6a28c91277e2204e1b9c33ae412faa26137b1c609886540526a68aac
-  data.tar.gz: 01f89de5cb1cbc2f94cbeb82b996bcabd5447b7d2791c985070a8cd053ef936cc961d7d7d4b17902ceda923e870a8c572e6fae55c4148c510eba52e65a931cb9
+  metadata.gz: dd936cd55b94200bb344d78236527f31cb298e2a453ca660f4d825ba0538c9d6057e730b975319f68130fe64d847cdacb7937a4dbb0344f3ceaf43efb7f3b7b0
+  data.tar.gz: 3d8814c197319adb5fb8e9fb2e1c5b5d46dee8adb3bab204199d98b78137b9cbdf8ca28873405b65af8c570ff499a93981851a2bd5e962f1c5910c9303c00125

data/.reek.yml

@@ -21,18 +21,21 @@ detectors:
   Attribute:
     exclude:
       - Truemail::Configuration#smtp_safe_check
-      - Truemail::Validate::ResolverExecutionWrapper#attempts
+      - Truemail::Wrapper#attempts
 
   UtilityFunction:
     exclude:
       - Truemail::Validate::Smtp::Request#compose_from
       - Truemail::Validator#select_validation_type
       - Truemail::Validate::Mx#null_mx?
+      - Truemail::Validate::Mx#a_record
+      - Truemail::Audit::Ptr#current_host_address
 
   ControlParameter:
     exclude:
       - Truemail::GenerateEmailHelper#calculate_email_size
-      - Truemail::Validate::Base#success
+      - Truemail::Worker#success
+      - Truemail#raise_unless
 
   FeatureEnvy:
     exclude:

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    truemail (0.1.6)
+    truemail (0.1.7)
 
 GEM
   remote: https://rubygems.org/

data/README.md

@@ -2,7 +2,7 @@
 
 [![Maintainability](https://api.codeclimate.com/v1/badges/657aa241399927dcd2e2/maintainability)](https://codeclimate.com/github/rubygarage/truemail/maintainability) [![Test Coverage](https://api.codeclimate.com/v1/badges/657aa241399927dcd2e2/test_coverage)](https://codeclimate.com/github/rubygarage/truemail/test_coverage) [![Gem Version](https://badge.fury.io/rb/truemail.svg)](https://badge.fury.io/rb/truemail) [![CircleCI](https://circleci.com/gh/rubygarage/truemail/tree/master.svg?style=svg)](https://circleci.com/gh/rubygarage/truemail/tree/master) [![Contributor Covenant](https://img.shields.io/badge/Contributor%20Covenant-v1.4%20adopted-ff69b4.svg)](CODE_OF_CONDUCT.md)
 
-The Truemail gem helps you validate emails by regex pattern, presence of domain mx-records, and real existence of email account on a current email server.
+The Truemail gem helps you validate emails by regex pattern, presence of domain mx-records, and real existence of email account on a current email server. Also Truemail gem allows performing an audit of the host in which runs.
 
 ## Features
 
@@ -137,7 +137,7 @@ Truemail.configuration
 
 #### Regex validation
 
-Validation with regex pattern is the first validation level. By default this validation not performs strictly following RFC 5322 standart, so you can override Truemail default regex pattern if you want.
+Validation with regex pattern is the first validation level. By default this validation not performs strictly following RFC 5322 standard, so you can override Truemail default regex pattern if you want.
 
 Example of usage:
 
@@ -383,6 +383,30 @@ Truemail.validate('email@example.com')
     @validation_type=:smtp>
 ```
 
+### Host audit features
+
+Truemail gem allows performing an audit of the host in which runs. Only PTR record audit performs for today.
+
+#### PTR audit
+
+So what is a PTR record? A PTR record, or pointer record, enables someone to perform a reverse DNS lookup. This allows them to determine your domain name based on your IP address. Because generic domain names without a PTR are often associated with spammers, incoming mail servers identify email from hosts without PTR records as spam and you can't verify yours emails qualitatively.
+
+```ruby
+Truemail.host_audit
+# Everything is good
+=> #<Truemail::Auditor:0x00005580df358828
+   @result=
+     #<struct Truemail::Auditor::Result
+       warnings={}>>
+
+# Has PTR warning
+=> #<Truemail::Auditor:0x00005580df358828
+   @result=
+     #<struct Truemail::Auditor::Result
+       warnings=
+         {:ptr=>"ptr record does not reference to current verifier domain"}>>
+```
+
 ### Truemail helpers
 
 #### .valid?

data/lib/truemail.rb

@@ -1,9 +1,6 @@
 # frozen_string_literal: true
 
-require 'truemail/version'
 require 'truemail/core'
-require 'truemail/configuration'
-require 'truemail/validator'
 
 module Truemail
   INCOMPLETE_CONFIG = 'verifier_email is required parameter'
@@ -15,7 +12,7 @@ module Truemail
         return unless block_given?
         configuration = Truemail::Configuration.new
         yield(configuration)
-        raise ConfigurationError, INCOMPLETE_CONFIG unless configuration.complete?
+        raise_unless(configuration.complete?, INCOMPLETE_CONFIG)
         configuration
       end
     end
@@ -29,12 +26,23 @@ module Truemail
     end
 
     def validate(email, **options)
-      raise ConfigurationError, NOT_CONFIGURED unless configuration
+      raise_unless(configuration, NOT_CONFIGURED)
       Truemail::Validator.new(email, **options).run
     end
 
     def valid?(email, **options)
       validate(email, **options).result.valid?
     end
+
+    def host_audit
+      raise_unless(configuration, NOT_CONFIGURED)
+      Truemail::Auditor.run
+    end
+
+    private
+
+    def raise_unless(condition, message)
+      raise ConfigurationError, message unless condition
+    end
   end
 end

data/lib/truemail/audit/base.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+module Truemail
+  module Audit
+    class Base < Truemail::Worker
+      private
+
+      def add_warning(message)
+        result.warnings[self.class.name.split('::').last.downcase.to_sym] = message
+      end
+    end
+  end
+end

data/lib/truemail/audit/ptr.rb

@@ -0,0 +1,41 @@
+# frozen_string_literal: true
+
+module Truemail
+  module Audit
+    class Ptr < Truemail::Audit::Base
+      require 'ipaddr'
+      require 'resolv'
+
+      NOT_FOUND = 'ptr record for current host address was not found'
+      NOT_REFERENCES = 'ptr record does not reference to current verifier domain'
+
+      def run
+        return if ptr_records.empty? && add_warning(Truemail::Audit::Ptr::NOT_FOUND)
+        return if ptr_references_to_verifier_domain?
+        add_warning(Truemail::Audit::Ptr::NOT_REFERENCES)
+      end
+
+      private
+
+      def current_host_address
+        Resolv.getaddress(Socket.gethostname)
+      end
+
+      def current_host_reverse_lookup
+        IPAddr.new(current_host_address).reverse
+      end
+
+      def ptr_records
+        @ptr_records ||= Truemail::Wrapper.call do
+          Resolv::DNS.new.getresources(
+            current_host_reverse_lookup, Resolv::DNS::Resource::IN::PTR
+          ).map { |ptr_record| ptr_record.name.to_s }
+        end || []
+      end
+
+      def ptr_references_to_verifier_domain?
+        ptr_records.include?(Truemail.configuration.verifier_domain)
+      end
+    end
+  end
+end

data/lib/truemail/auditor.rb

@@ -0,0 +1,24 @@
+# frozen_string_literal: true
+
+module Truemail
+  class Auditor
+    Result = Struct.new(:warnings, keyword_init: true) do
+      def initialize(warnings: {}, **args)
+        super
+      end
+    end
+
+    def self.run
+      new.run
+    end
+
+    def result
+      @result ||= Truemail::Auditor::Result.new
+    end
+
+    def run
+      Truemail::Audit::Ptr.check(result)
+      self
+    end
+  end
+end

data/lib/truemail/core.rb

@@ -1,6 +1,13 @@
 # frozen_string_literal: true
 
 module Truemail
+  require 'truemail/version'
+  require 'truemail/configuration'
+  require 'truemail/worker'
+  require 'truemail/wrapper'
+  require 'truemail/auditor'
+  require 'truemail/validator'
+
   class ConfigurationError < StandardError; end
 
   class ArgumentError < StandardError
@@ -16,10 +23,14 @@ module Truemail
     REGEX_DOMAIN_FROM_EMAIL = /\A.+@(.+)\z/
   end
 
+  module Audit
+    require 'truemail/audit/base'
+    require 'truemail/audit/ptr'
+  end
+
   module Validate
     require 'truemail/validate/base'
     require 'truemail/validate/regex'
-    require 'truemail/validate/resolver_execution_wrapper'
     require 'truemail/validate/mx'
     require 'truemail/validate/smtp'
     require 'truemail/validate/smtp/response'

data/lib/truemail/validate/base.rb

@@ -2,23 +2,9 @@
 
 module Truemail
   module Validate
-    class Base
-      attr_reader :result
-
-      def self.check(result)
-        new(result).run
-      end
-
-      def initialize(result)
-        @result = result
-      end
-
+    class Base < Truemail::Worker
       private
 
-      def success(condition)
-        result.success = condition || false
-      end
-
       def add_error(message)
         result.errors[self.class.name.split('::').last.downcase.to_sym] = message
       end

data/lib/truemail/validate/mx.rb

@@ -24,7 +24,7 @@ module Truemail
 
       def mx_lookup
         host_extractor_methods.any? do |method|
-          Truemail::Validate::ResolverExecutionWrapper.call { send(method) }
+          Truemail::Wrapper.call { send(method) }
         end
       end
 
@@ -41,8 +41,8 @@ module Truemail
         !mail_servers.include?(Truemail::Validate::Mx::NULL_MX_RECORD)
       end
 
-      def mx_records(domain)
-        domain_mx_records = Resolv::DNS.new.getresources(domain, Resolv::DNS::Resource::IN::MX)
+      def mx_records(hostname)
+        domain_mx_records = Resolv::DNS.new.getresources(hostname, Resolv::DNS::Resource::IN::MX)
         return [Truemail::Validate::Mx::NULL_MX_RECORD] if null_mx?(domain_mx_records)
         domain_mx_records.sort_by(&:preference).map do |mx_record|
           Resolv.getaddresses(mx_record.exchange.to_s)
@@ -53,16 +53,24 @@ module Truemail
         !mail_servers.empty?
       end
 
+      def domain
+        result.domain
+      end
+
       def hosts_from_mx_records?
-        fetch_target_hosts(mx_records(result.domain))
+        fetch_target_hosts(mx_records(domain))
         mail_servers_found?
       end
 
+      def a_record(hostname)
+        Resolv.getaddress(hostname)
+      end
+
       def hosts_from_cname_records?
-        cname_records = Resolv::DNS.new.getresources(result.domain, Resolv::DNS::Resource::IN::CNAME)
+        cname_records = Resolv::DNS.new.getresources(domain, Resolv::DNS::Resource::IN::CNAME)
         return if cname_records.empty?
         cname_records.each do |cname_record|
-          host = Resolv.getaddress(cname_record.name.to_s)
+          host = a_record(cname_record.name.to_s)
           hostname = Resolv.getname(host)
           found_hosts = mx_records(hostname)
           fetch_target_hosts(found_hosts.empty? ? [host] : found_hosts)
@@ -71,7 +79,7 @@ module Truemail
       end
 
       def host_from_a_record?
-        fetch_target_hosts([Resolv.getaddress(result.domain)])
+        fetch_target_hosts([a_record(domain)])
         mail_servers_found?
       end
     end

data/lib/truemail/validate/smtp.rb

@@ -4,7 +4,7 @@ module Truemail
   module Validate
     class Smtp < Truemail::Validate::Base
       ERROR = 'smtp error'
-      ERROR_BODY = /(?=.*550)(?=.*(user|account)).*/i
+      ERROR_BODY = /(?=.*550)(?=.*(user|account|customer|mailbox)).*/i
 
       attr_reader :smtp_results
 

data/lib/truemail/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Truemail
-  VERSION = '0.1.6'
+  VERSION = '0.1.7'
 end

data/lib/truemail/worker.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+module Truemail
+  class Worker
+    attr_reader :result
+
+    def self.check(result)
+      new(result).run
+    end
+
+    def initialize(result)
+      @result = result
+    end
+
+    private
+
+    def success(condition)
+      result.success = condition || false
+    end
+  end
+end

data/lib/truemail/wrapper.rb

@@ -0,0 +1,24 @@
+# frozen_string_literal: true
+
+module Truemail
+  class Wrapper
+    attr_accessor :attempts
+
+    def self.call(&block)
+      new.call(&block)
+    end
+
+    def initialize
+      @attempts = Truemail.configuration.connection_attempts
+    end
+
+    def call(&block)
+      Timeout.timeout(Truemail.configuration.connection_timeout, &block)
+    rescue Resolv::ResolvError
+      false
+    rescue Timeout::Error
+      retry unless (self.attempts -= 1).zero?
+      false
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: truemail
 version: !ruby/object:Gem::Version
-  version: 0.1.6
+  version: 0.1.7
 platform: ruby
 authors:
 - Vladislav Trotsenko
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2019-04-08 00:00:00.000000000 Z
+date: 2019-04-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -193,17 +193,21 @@ files:
 - bin/console
 - bin/setup
 - lib/truemail.rb
+- lib/truemail/audit/base.rb
+- lib/truemail/audit/ptr.rb
+- lib/truemail/auditor.rb
 - lib/truemail/configuration.rb
 - lib/truemail/core.rb
 - lib/truemail/validate/base.rb
 - lib/truemail/validate/mx.rb
 - lib/truemail/validate/regex.rb
-- lib/truemail/validate/resolver_execution_wrapper.rb
 - lib/truemail/validate/smtp.rb
 - lib/truemail/validate/smtp/request.rb
 - lib/truemail/validate/smtp/response.rb
 - lib/truemail/validator.rb
 - lib/truemail/version.rb
+- lib/truemail/worker.rb
+- lib/truemail/wrapper.rb
 - truemail.gemspec
 homepage: https://github.com/rubygarage/truemail
 licenses:

data/lib/truemail/validate/resolver_execution_wrapper.rb

@@ -1,26 +0,0 @@
-# frozen_string_literal: true
-
-module Truemail
-  module Validate
-    class ResolverExecutionWrapper
-      attr_accessor :attempts
-
-      def self.call(&block)
-        new.call(&block)
-      end
-
-      def initialize
-        @attempts = Truemail.configuration.connection_attempts
-      end
-
-      def call(&block)
-        Timeout.timeout(Truemail.configuration.connection_timeout, &block)
-      rescue Resolv::ResolvError
-        false
-      rescue Timeout::Error
-        retry unless (self.attempts -= 1).zero?
-        false
-      end
-    end
-  end
-end