truemail 2.4.7 → 2.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 29ccfdfd6a113d27d58b10152fdcad9b36776483f218913d1adb8ea4620734e7
-  data.tar.gz: 03a3359ff3f9802294232a3c044803079d0ae8827231290b922ef986bd65e3a5
+  metadata.gz: 1cdc3f36219adee744f6bcbbdd45ff4fe5e9e7f76096ff01dacd2f7c8f192d91
+  data.tar.gz: f8e40d25bf15b1edf3f5a1dde85fa02913947087f962382f85b975d42dde58f8
 SHA512:
-  metadata.gz: 75641c556bde056926b6a2e72c7f050ebb0abb15e0a383260e29e82ec2115dfa5510992a2931e62a13dc98a6c1704eeaf3e038950c73353c88466e5eeba8d73f
-  data.tar.gz: 77876e0bb2cf55c82fded8c9c5c746395d5909ab1ddc8cff2e751d3f731bcaeb51ccd449b0792f096e902b8762f81a2f0ed32f339d8ef149f0ddf4d6f7a4c21c
+  metadata.gz: ce97f45d8128d2096a1f06e8afb5078cd6b53768061d9e0c8c86be91228e69c50cb4f0151f099eddef08f22192aeb6f792289e86af7d8d8dac185398b6a56f23
+  data.tar.gz: 209d6746e691dae8eba6b0571b67ffff51947106ee85f7e415ce4d0acd5efe871b1b1de23528ec8130fe08800256e1f8b7bf54aee2bf18b6ea5e0a22d328814e

data/.codeclimate.yml

@@ -7,7 +7,7 @@ checks:
 plugins:
   rubocop:
     enabled: true
-    channel: rubocop-1-18
+    channel: rubocop-1-22
 
   reek:
     enabled: true

data/.rubocop.yml

@@ -166,6 +166,18 @@ Style/MultilineInPatternThen:
 Style/QuotedSymbols:
   Enabled: true
 
+Style/RedundantSelfAssignmentBranch:
+  Enabled: true
+
+Style/NumberedParameters:
+  Enabled: true
+
+Style/NumberedParametersLimit:
+  Enabled: true
+
+Style/SelectByRegexp:
+  Enabled: true
+
 Layout/LineLength:
   Max: 140
 
@@ -326,9 +338,24 @@ Lint/TripleQuotes:
 Lint/EmptyInPattern:
   Enabled: true
 
+Lint/AmbiguousRange:
+  Enabled: true
+
+Lint/AmbiguousOperatorPrecedence:
+  Enabled: true
+
+Lint/IncompatibleIoSelectWithFiberScheduler:
+  Enabled: true
+
+Lint/RequireRelativeSelfPath:
+  Enabled: true
+
 Gemspec/DateAssignment:
   Enabled: true
 
+Security/IoMethods:
+  Enabled: true
+
 Performance/AncestorsInclude:
   Enabled: true
 
@@ -412,3 +439,9 @@ RSpec/IdenticalEqualityAssertion:
 
 RSpec/Rails/AvoidSetupHook:
   Enabled: true
+
+RSpec/ExcessiveDocstringSpacing:
+  Enabled: true
+
+RSpec/SubjectDeclaration:
+  Enabled: true

data/CHANGELOG.md

@@ -2,12 +2,49 @@
 
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/), and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [2.5.1] - 2021.10.01
+
+### Updated
+
+- Updated gem development dependencies
+- Updated gem version
+
+## [2.5.0] - 2021.09.01
+
+### Updated
+
+Optimized DNS (MX) validation flow. Removed needless DNS request for case when custom email pattern was defined and email for validation includes invalid domain name.
+
+- Updated `Truemail::Validate::Mx#run`, tests
+- Updated gem development dependencies
+- Updated gem documentation, version
+
+## [2.4.9] - 2021.08.20
+
+### Updated
+
+- Updated `Truemail::Validate::DomainListMatch#email_domain`, tests
+- Updated `Truemail::Validate::Mx#domain`, tests
+- Updated gem development dependencies
+- Updated gem version
+
+## [2.4.8] - 2021.08.12
+
+### Updated
+
+- Updated gem development dependencies
+- Updated gem version
+
+### Changed
+
+- `faker` to `ffaker` development dependency
+
 ## [2.4.7] - 2021.08.09
 
 ### Updated
 
 - Updated gem codebase, refactored `Truemail::ContextHelper`
-- Updated tests
+- Updated tests with `Truemail::DnsHelper#dns_mock_gateway`
 - Updated gem development dependencies
 - Updated gem version
 

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    truemail (2.4.7)
+    truemail (2.5.1)
       simpleidn (~> 0.2.1)
 
 GEM
@@ -10,28 +10,24 @@ GEM
     addressable (2.8.0)
       public_suffix (>= 2.0.2, < 5.0)
     ast (2.4.2)
-    bundler-audit (0.8.0)
+    bundler-audit (0.9.0.1)
       bundler (>= 1.2.0, < 3)
       thor (~> 1.0)
     byebug (11.1.3)
     childprocess (4.1.0)
     coderay (1.1.3)
     colorize (0.8.1)
-    concurrent-ruby (1.1.9)
     crack (0.4.5)
       rexml
     diff-lcs (1.4.4)
-    dns_mock (1.4.0)
+    dns_mock (1.4.2)
       simpleidn (~> 0.2.1)
     docile (1.4.0)
-    faker (2.18.0)
-      i18n (>= 1.6, < 2)
     fasterer (0.9.0)
       colorize (~> 0.7)
       ruby_parser (>= 3.14.1)
+    ffaker (2.19.0)
     hashdiff (1.0.1)
-    i18n (1.8.10)
-      concurrent-ruby (~> 1.0)
     iniparse (1.5.0)
     json (2.5.1)
     json_matchers (0.11.1)
@@ -43,7 +39,7 @@ GEM
       childprocess (>= 0.6.3, < 5)
       iniparse (~> 1.4)
       rexml (~> 3.2)
-    parallel (1.20.1)
+    parallel (1.21.0)
     parser (3.0.2.0)
       ast (~> 2.4.1)
     pry (0.13.1)
@@ -52,14 +48,12 @@ GEM
     pry-byebug (3.9.0)
       byebug (~> 11.0)
       pry (~> 0.13.0)
-    psych (3.3.2)
     public_suffix (4.0.6)
     rainbow (3.0.0)
     rake (13.0.6)
-    reek (6.0.5)
+    reek (6.0.6)
       kwalify (~> 0.7.0)
       parser (~> 3.0.0)
-      psych (>= 3.1, < 5.0)
       rainbow (>= 2.0, < 4.0)
     regexp_parser (2.1.1)
     rexml (3.2.5)
@@ -76,23 +70,22 @@ GEM
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.10.0)
     rspec-support (3.10.2)
-    rubocop (1.18.4)
+    rubocop (1.22.0)
       parallel (~> 1.10)
       parser (>= 3.0.0.0)
       rainbow (>= 2.2.2, < 4.0)
       regexp_parser (>= 1.8, < 3.0)
       rexml
-      rubocop-ast (>= 1.8.0, < 2.0)
+      rubocop-ast (>= 1.12.0, < 2.0)
       ruby-progressbar (~> 1.7)
       unicode-display_width (>= 1.4.0, < 3.0)
-    rubocop-ast (1.9.0)
+    rubocop-ast (1.12.0)
       parser (>= 3.0.1.1)
-    rubocop-performance (1.11.4)
+    rubocop-performance (1.11.5)
       rubocop (>= 1.7.0, < 2.0)
       rubocop-ast (>= 0.4.0)
-    rubocop-rspec (2.4.0)
-      rubocop (~> 1.0)
-      rubocop-ast (>= 1.1.0)
+    rubocop-rspec (2.5.0)
+      rubocop (~> 1.19)
     ruby-progressbar (1.11.0)
     ruby_parser (3.17.0)
       sexp_processor (~> 4.15, >= 4.15.1)
@@ -105,14 +98,14 @@ GEM
     simpleidn (0.2.1)
       unf (~> 0.1.4)
     thor (1.1.0)
-    truemail-rspec (0.6.0)
-      faker (~> 2.17)
+    truemail-rspec (0.7.0)
+      ffaker (~> 2.18)
       rspec (~> 3.10)
       truemail (>= 2.4)
     unf (0.1.4)
       unf_ext
-    unf_ext (0.0.7.7)
-    unicode-display_width (2.0.0)
+    unf_ext (0.0.8)
+    unicode-display_width (2.1.0)
     webmock (3.14.0)
       addressable (>= 2.8.0)
       crack (>= 0.3.2)
@@ -122,24 +115,24 @@ PLATFORMS
   x86_64-darwin-20
 
 DEPENDENCIES
-  bundler (~> 2.2, >= 2.2.25)
-  bundler-audit (~> 0.8.0)
-  dns_mock (~> 1.4)
-  faker (~> 2.18)
+  bundler (~> 2.2, >= 2.2.28)
+  bundler-audit (~> 0.9.0.1)
+  dns_mock (~> 1.4, >= 1.4.2)
   fasterer (~> 0.9.0)
+  ffaker (~> 2.19)
   json_matchers (~> 0.11.1)
   overcommit (~> 0.58.0)
   pry-byebug (~> 3.9)
   rake (~> 13.0, >= 13.0.6)
-  reek (~> 6.0, >= 6.0.5)
+  reek (~> 6.0, >= 6.0.6)
   rspec (~> 3.10)
-  rubocop (~> 1.18, >= 1.18.4)
-  rubocop-performance (~> 1.11, >= 1.11.4)
-  rubocop-rspec (~> 2.4)
+  rubocop (~> 1.22)
+  rubocop-performance (~> 1.11, >= 1.11.5)
+  rubocop-rspec (~> 2.5)
   simplecov (~> 0.17.1)
   truemail!
-  truemail-rspec (~> 0.6)
+  truemail-rspec (~> 0.7.0)
   webmock (~> 3.14)
 
 BUNDLED WITH
-   2.2.25
+   2.2.28

data/README.md

@@ -368,7 +368,7 @@ Truemail.validate('email@white-domain.com')
   @result=#<struct Truemail::Validator::Result
     success=true,
     email="email@white-domain.com",
-    domain=nil,
+    domain="white-domain.com",
     mail_servers=[],
     errors={},
     smtp_debug=nil>,
@@ -416,7 +416,7 @@ Truemail.validate('email@white-domain.com', with: :regex)
   @result=#<struct Truemail::Validator::Result
     success=true,
     email="email@white-domain.com",
-    domain=nil,
+    domain="white-domain.com",
     mail_servers=[],
     errors={},
     smtp_debug=nil>,
@@ -451,7 +451,7 @@ Truemail.validate('email@domain.com', with: :regex)
   @result=#<struct Truemail::Validator::Result
     success=false,
     email="email@domain.com",
-    domain=nil,
+    domain="domain.com",
     mail_servers=[],
     errors={},
     smtp_debug=nil>,
@@ -488,7 +488,7 @@ Truemail.validate('email@black-domain.com')
   @result=#<struct Truemail::Validator::Result
     success=false,
     email="email@black-domain.com",
-    domain=nil,
+    domain="black-domain.com",
     mail_servers=[],
     errors={},
     smtp_debug=nil>,
@@ -525,7 +525,7 @@ Truemail.validate('email@somedomain.com')
   @result=#<struct Truemail::Validator::Result
     success=true,
     email="email@somedomain.com",
-    domain=nil,
+    domain="somedomain.com",
     mail_servers=[],
     errors={},
     smtp_debug=nil>,
@@ -579,7 +579,7 @@ Truemail.validate('email@example.com', with: :regex)
     #<struct Truemail::Validator::Result
       success=true,
       email="email@example.com",
-      domain=nil,
+      domain="example.com",
       mail_servers=[],
       errors={},
       smtp_debug=nil>,
@@ -624,7 +624,7 @@ Truemail.validate('email@example.com', with: :regex)
     #<struct Truemail::Validator::Result
       success=true,
       email="email@example.com",
-      domain=nil,
+      domain="example.com",
       mail_servers=[],
       errors={},
       smtp_debug=nil>,

data/lib/truemail/dns/punycode_representer.rb

@@ -2,7 +2,7 @@
 
 module Truemail
   module Dns
-    PunycodeRepresenter = Class.new do
+    PunycodeRepresenter = ::Class.new do
       require 'simpleidn'
 
       def self.call(email)

data/lib/truemail/validate/domain_list_match.rb

@@ -15,7 +15,8 @@ module Truemail
       private
 
       def email_domain
-        @email_domain ||= result.email[Truemail::RegexConstant::REGEX_DOMAIN_FROM_EMAIL, 1]
+        @email_domain ||=
+          result.domain = result.email[Truemail::RegexConstant::REGEX_DOMAIN_FROM_EMAIL, 1]
       end
 
       def whitelisted_domain?

data/lib/truemail/validate/mx.rb

@@ -8,13 +8,17 @@ module Truemail
 
       def run
         return false unless Truemail::Validate::Regex.check(result)
-        return true if success(mx_lookup && domain_not_include_null_mx)
+        return true if success(email_domain_valid? && mx_lookup && domain_not_include_null_mx)
         mail_servers.clear && add_error(Truemail::Validate::Mx::ERROR)
         false
       end
 
       private
 
+      def email_domain_valid?
+        Truemail::RegexConstant::REGEX_DOMAIN_PATTERN.match?(result.domain)
+      end
+
       def host_extractor_methods
         return %i[hosts_from_mx_records?] if configuration.not_rfc_mx_lookup_flow
         %i[hosts_from_mx_records? hosts_from_cname_records? host_from_a_record?]
@@ -52,10 +56,7 @@ module Truemail
       end
 
       def domain
-        @domain ||= begin
-          result.domain = result.email[Truemail::RegexConstant::REGEX_DOMAIN_FROM_EMAIL, 1]
-          result.punycode_email[Truemail::RegexConstant::REGEX_DOMAIN_FROM_EMAIL, 1]
-        end
+        @domain ||= result.punycode_email[Truemail::RegexConstant::REGEX_DOMAIN_FROM_EMAIL, 1]
       end
 
       def hosts_from_mx_records?

data/lib/truemail/validate/smtp.rb

@@ -35,7 +35,7 @@ module Truemail
       def attempts
         @attempts ||= begin
           return {} if fail_fast? || !mail_servers.one?
-          { attempts: configuration.connection_attempts }
+          { attempts: configuration.connection_attempts } # attempts should used for case with one mail server only
         end
       end
 

data/lib/truemail/version.rb

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

data/truemail.gemspec

@@ -28,26 +28,26 @@ Gem::Specification.new do |spec|
 
   spec.files = `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
   spec.bindir        = 'exe'
-  spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
+  spec.executables   = spec.files.grep(%r{^exe/}) { |f| ::File.basename(f) }
   spec.require_paths = ['lib']
 
   spec.add_runtime_dependency 'simpleidn', '~> 0.2.1'
 
-  spec.add_development_dependency 'bundler', '~> 2.2', '>= 2.2.25'
-  spec.add_development_dependency 'bundler-audit', '~> 0.8.0'
-  spec.add_development_dependency 'dns_mock', '~> 1.4'
-  spec.add_development_dependency 'faker', '~> 2.18'
+  spec.add_development_dependency 'bundler', '~> 2.2', '>= 2.2.28'
+  spec.add_development_dependency 'bundler-audit', '~> 0.9.0.1'
+  spec.add_development_dependency 'dns_mock', '~> 1.4', '>= 1.4.2'
   spec.add_development_dependency 'fasterer', '~> 0.9.0'
+  spec.add_development_dependency 'ffaker', '~> 2.19'
   spec.add_development_dependency 'json_matchers', '~> 0.11.1'
   spec.add_development_dependency 'overcommit', '~> 0.58.0'
   spec.add_development_dependency 'pry-byebug', '~> 3.9'
   spec.add_development_dependency 'rake', '~> 13.0', '>= 13.0.6'
-  spec.add_development_dependency 'reek', '~> 6.0', '>= 6.0.5'
+  spec.add_development_dependency 'reek', '~> 6.0', '>= 6.0.6'
   spec.add_development_dependency 'rspec', '~> 3.10'
-  spec.add_development_dependency 'rubocop', '~> 1.18', '>= 1.18.4'
-  spec.add_development_dependency 'rubocop-performance', '~> 1.11', '>= 1.11.4'
-  spec.add_development_dependency 'rubocop-rspec', '~> 2.4'
+  spec.add_development_dependency 'rubocop', '~> 1.22'
+  spec.add_development_dependency 'rubocop-performance', '~> 1.11', '>= 1.11.5'
+  spec.add_development_dependency 'rubocop-rspec', '~> 2.5'
   spec.add_development_dependency 'simplecov', '~> 0.17.1'
-  spec.add_development_dependency 'truemail-rspec', '~> 0.6'
+  spec.add_development_dependency 'truemail-rspec', '~> 0.7.0'
   spec.add_development_dependency 'webmock', '~> 3.14'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: truemail
 version: !ruby/object:Gem::Version
-  version: 2.4.7
+  version: 2.5.1
 platform: ruby
 authors:
 - Vladislav Trotsenko
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2021-08-09 00:00:00.000000000 Z
+date: 2021-10-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: simpleidn
@@ -33,7 +33,7 @@ dependencies:
         version: '2.2'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 2.2.25
+        version: 2.2.28
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -43,21 +43,21 @@ dependencies:
         version: '2.2'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 2.2.25
+        version: 2.2.28
 - !ruby/object:Gem::Dependency
   name: bundler-audit
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.8.0
+        version: 0.9.0.1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.8.0
+        version: 0.9.0.1
 - !ruby/object:Gem::Dependency
   name: dns_mock
   requirement: !ruby/object:Gem::Requirement
@@ -65,6 +65,9 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.4'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.4.2
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -72,34 +75,37 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.4'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.4.2
 - !ruby/object:Gem::Dependency
-  name: faker
+  name: fasterer
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.18'
+        version: 0.9.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.18'
+        version: 0.9.0
 - !ruby/object:Gem::Dependency
-  name: fasterer
+  name: ffaker
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.9.0
+        version: '2.19'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.9.0
+        version: '2.19'
 - !ruby/object:Gem::Dependency
   name: json_matchers
   requirement: !ruby/object:Gem::Requirement
@@ -171,7 +177,7 @@ dependencies:
         version: '6.0'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 6.0.5
+        version: 6.0.6
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -181,7 +187,7 @@ dependencies:
         version: '6.0'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 6.0.5
+        version: 6.0.6
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -202,20 +208,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.18'
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 1.18.4
+        version: '1.22'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.18'
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 1.18.4
+        version: '1.22'
 - !ruby/object:Gem::Dependency
   name: rubocop-performance
   requirement: !ruby/object:Gem::Requirement
@@ -225,7 +225,7 @@ dependencies:
         version: '1.11'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.11.4
+        version: 1.11.5
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -235,21 +235,21 @@ dependencies:
         version: '1.11'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.11.4
+        version: 1.11.5
 - !ruby/object:Gem::Dependency
   name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.4'
+        version: '2.5'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.4'
+        version: '2.5'
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
@@ -270,14 +270,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.6'
+        version: 0.7.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.6'
+        version: 0.7.0
 - !ruby/object:Gem::Dependency
   name: webmock
   requirement: !ruby/object:Gem::Requirement