truemail 1.1.0 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 20487757313dd1f93984687a1bb9190b5a8e7a9f619d41a5ae0b550230de63a5
-  data.tar.gz: fc67f8aaa93041388ff21a9dfb3cbeee16581929cd8217cb2101de64ef2c5af7
+  metadata.gz: 0dcc1349ffc22296ae3af4dd04ce6b42501964ff1ca2072791d3e41320478d38
+  data.tar.gz: 26e29784585ca4ca560f574d7cf60f23e9aa9300757f11a1567881efa826b53a
 SHA512:
-  metadata.gz: 2f4551903201185b354ec56c919fba276633da90546a8de1c4a3e57d0f4b786de6b1d7c667be61af5a46414569f9d00c98d19e0cc9e3bfa23c80c807fc38ba1e
-  data.tar.gz: 27873755f8df550ce1b2be550b8ad6650a3b4dbbeb1f5d0ee7bade7588d2d7a0b3eb1fe539a596ab05ecf220391006f6d10bfb95dfb312ce05653b2c59bf5c7a
+  metadata.gz: b51544bb6b74fe12510f1e008c1a2265ad4eb9fed77a4618025bccc22571c748d99d02ca8994504576b0011ca6064469f94b77c56f1d6dd4ed896cbb1c3f1929
+  data.tar.gz: 260c8f3c1b2a7d0fdce382d325a510187e1c92ec7d658f2c12e04ef231186ed49a433de826fec5f2ef9c6a30076ba04abd8f29636a10cee2a25711ad0337178d

data/.github/ISSUE_TEMPLATE.md

@@ -3,8 +3,8 @@
 ### New Issue Checklist
 
 - [ ] I have updated truemail to the latest version
-- [ ] I have read the [Contribution Guidelines](../CONTRIBUTING.md)
-- [ ] I have read the [documentation](../README.md)
+- [ ] I have read the [Contribution Guidelines](https://github.com/rubygarage/truemail/blob/master/CONTRIBUTING.md)
+- [ ] I have read the [documentation](https://github.com/rubygarage/truemail/blob/master/README.md)
 - [ ] I have searched for [existing GitHub issues](https://github.com/rubygarage/truemail/issues)
 
 ### Issue Description

data/.github/PULL_REQUEST_TEMPLATE.md

@@ -40,7 +40,7 @@
 - [ ] My code follows the code style of this project
 - [ ] My change requires a change to the documentation
 - [ ] I have updated the documentation accordingly
-- [ ] I have read the [**CONTRIBUTING** document](../CONTRIBUTING.md)
+- [ ] I have read the [**CONTRIBUTING** document](https://github.com/rubygarage/truemail/blob/master/CONTRIBUTING.md)
 - [ ] I have added tests to cover my changes
 - [ ] I have run `bundle exec rspec` from the root directory to see all new and existing tests pass
 - [ ] I have run `rubocop` and `reek` to ensure the code style is valid

data/.reek.yml

@@ -20,6 +20,7 @@ detectors:
 
   Attribute:
     exclude:
+      - Truemail::Configuration#whitelist_validation
       - Truemail::Configuration#smtp_safe_check
       - Truemail::Wrapper#attempts
 

data/.rubocop.yml

@@ -7,6 +7,9 @@ AllCops:
 Metrics/LineLength:
   Max: 140
 
+Metrics/MethodLength:
+  Max: 15
+
 Metrics/CyclomaticComplexity:
   Enabled: false
 

data/Gemfile.lock

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

data/README.md

@@ -1,6 +1,6 @@
 # Truemail
 
-[![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)
+[![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) [![CircleCI](https://circleci.com/gh/rubygarage/truemail/tree/master.svg?style=svg)](https://circleci.com/gh/rubygarage/truemail/tree/master) [![Gem Version](https://badge.fury.io/rb/truemail.svg)](https://badge.fury.io/rb/truemail) [![Downloads](https://img.shields.io/gem/dt/truemail.svg?colorA=004d99&colorB=0073e6)](https://rubygems.org/gems/truemail) [![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. Also Truemail gem allows performing an audit of the host in which runs.
 
@@ -90,7 +90,12 @@ Truemail.configure do |config|
   # return true. Other validations will not processed even if it was defined in validation_type_for
   config.whitelisted_domains = ['somedomain1.com', 'somedomain2.com']
 
-  # Optional parameter. Validation of email which contains whitelisted domain always will
+  # Optional parameter. With this option Truemail will validate email which contains whitelisted
+  # domain only, i.e. if domain whitelisted, validation will passed to Regex, MX or SMTP validators.
+  # Validation of email which not contains whitelisted domain always will return false.
+  config.whitelist_validation = true
+
+  # Optional parameter. Validation of email which contains blacklisted domain always will
   # return false. Other validations will not processed even if it was defined in validation_type_for
   config.blacklisted_domains = ['somedomain1.com', 'somedomain2.com']
 
@@ -116,6 +121,7 @@ Truemail.configuration
  @connection_attempts=3,
  @validation_type_by_domain={},
  @whitelisted_domains=[],
+ @whitelist_validation=true,
  @blacklisted_domains=[],
  @verifier_domain="somedomain.com",
  @verifier_email="verifier@example.com"
@@ -141,6 +147,7 @@ Truemail.configuration
  @connection_attempts=1,
  @validation_type_by_domain={},
  @whitelisted_domains=[],
+ @whitelist_validation=true,
  @blacklisted_domains=[],
  @verifier_domain="somedomain.com",
  @verifier_email="verifier@example.com",
@@ -168,7 +175,8 @@ Please note, other validations will not processed even if it was defined in ```v
 
 **Sequence of domain list check:**
 1. Whitelist check
-2. Blacklist check
+2. Whitelist validation check
+3. Blacklist check
 
 Example of usage:
 
@@ -201,6 +209,53 @@ Truemail.validate('email@white-domain.com')
   @validation_type=:whitelist>
 ```
 
+##### Whitelist validation case
+
+```ruby
+require 'truemail'
+
+Truemail.configure do |config|
+  config.verifier_email = 'verifier@example.com'
+  config.whitelisted_domains = ['white-domain.com']
+  config.whitelist_validation = true
+end
+```
+
+When email domain in whitelist and ```whitelist_validation``` is sets equal to ```true``` validation type will be passed to other validators.
+Validation of email which not contains whitelisted domain always will return ```false```.
+
+###### Email has whitelisted domain
+
+```ruby
+Truemail.validate('email@white-domain.com', with: :regex)
+
+#<Truemail::Validator:0x000055b8429f3490
+  @result=#<struct Truemail::Validator::Result
+    success=true,
+    email="email@white-domain.com",
+    domain=nil,
+    mail_servers=[],
+    errors={},
+    smtp_debug=nil>,
+  @validation_type=:regex>
+```
+
+###### Email hasn't whitelisted domain
+
+```ruby
+Truemail.validate('email@domain.com', with: :regex)
+
+#<Truemail::Validator:0x000055b8429f3490
+  @result=#<struct Truemail::Validator::Result
+    success=false,
+    email="email@domain.com",
+    domain=nil,
+    mail_servers=[],
+    errors={},
+    smtp_debug=nil>,
+  @validation_type=:blacklist>
+```
+
 ##### Blacklist case
 
 When email in blacklist, validation type will be redefined too. Validation result returns ```false```
@@ -384,6 +439,7 @@ Truemail.validate('email@example.com')
               @smtp_safe_check=false,
               @validation_type_by_domain={},
               @whitelisted_domains=[],
+              @whitelist_validation=false,
               @blacklisted_domains=[],
               @verifier_domain="example.com",
               @verifier_email="verifier@example.com">,
@@ -440,6 +496,7 @@ Truemail.validate('email@example.com')
                 @smtp_safe_check=true,
                 @validation_type_by_domain={},
                 @whitelisted_domains=[],
+                @whitelist_validation=false,
                 @blacklisted_domains=[],
                 @verifier_domain="example.com",
                 @verifier_email="verifier@example.com">,
@@ -480,6 +537,7 @@ Truemail.validate('email@example.com')
               @smtp_safe_check=true,
               @validation_type_by_domain={},
               @whitelisted_domains=[],
+              @whitelist_validation=false,
               @blacklisted_domains=[],
               @verifier_domain="example.com",
               @verifier_email="verifier@example.com">,
@@ -553,10 +611,6 @@ end
 ```
 
 ---
-## ToDo
-
-Fail validations logger
-
 ## Contributing
 
 Bug reports and pull requests are welcome on GitHub at https://github.com/rubygarage/truemail. This project is intended to be a safe, welcoming space for collaboration, and contributors are expected to adhere to the [Contributor Covenant](http://contributor-covenant.org) code of conduct. Please check the [open tikets](https://github.com/rubygarage/truemail/issues). Be shure to follow Contributor Code of Conduct below and our [Contributing Guidelines](CONTRIBUTING.md).

data/lib/truemail/configuration.rb

@@ -19,21 +19,14 @@ module Truemail
                 :whitelisted_domains,
                 :blacklisted_domains
 
-    attr_accessor :smtp_safe_check
+    attr_accessor :whitelist_validation, :smtp_safe_check
 
     alias retry_count connection_attempts
 
     def initialize
-      @email_pattern = Truemail::RegexConstant::REGEX_EMAIL_PATTERN
-      @smtp_error_body_pattern = Truemail::RegexConstant::REGEX_SMTP_ERROR_BODY_PATTERN
-      @connection_timeout = Truemail::Configuration::DEFAULT_CONNECTION_TIMEOUT
-      @response_timeout = Truemail::Configuration::DEFAULT_RESPONSE_TIMEOUT
-      @connection_attempts = Truemail::Configuration::DEFAULT_CONNECTION_ATTEMPTS
-      @default_validation_type = Truemail::Configuration::DEFAULT_VALIDATION_TYPE
-      @validation_type_by_domain = {}
-      @whitelisted_domains = []
-      @blacklisted_domains = []
-      @smtp_safe_check = false
+      instance_initializer.each do |instace_variable, value|
+        instance_variable_set(:"@#{instace_variable}", value)
+      end
     end
 
     %i[email_pattern smtp_error_body_pattern].each do |method|
@@ -84,6 +77,22 @@ module Truemail
 
     private
 
+    def instance_initializer
+      {
+        email_pattern: Truemail::RegexConstant::REGEX_EMAIL_PATTERN,
+        smtp_error_body_pattern: Truemail::RegexConstant::REGEX_SMTP_ERROR_BODY_PATTERN,
+        connection_timeout: Truemail::Configuration::DEFAULT_CONNECTION_TIMEOUT,
+        response_timeout: Truemail::Configuration::DEFAULT_RESPONSE_TIMEOUT,
+        connection_attempts: Truemail::Configuration::DEFAULT_CONNECTION_ATTEMPTS,
+        default_validation_type: Truemail::Configuration::DEFAULT_VALIDATION_TYPE,
+        validation_type_by_domain: {},
+        whitelisted_domains: [],
+        whitelist_validation: false,
+        blacklisted_domains: [],
+        smtp_safe_check: false
+      }
+    end
+
     def raise_unless(argument_context, argument_name, condition)
       raise Truemail::ArgumentError.new(argument_context, argument_name) unless condition
     end

data/lib/truemail/validate/domain_list_match.rb

@@ -6,8 +6,8 @@ module Truemail
       ERROR = 'blacklisted email'
 
       def run
-        return success(true) if whitelisted_domain?
-        return unless blacklisted_domain?
+        return success(true) if whitelisted_domain? && !whitelist_validation?
+        return unless whitelist_validation_case? || blacklisted_domain?
         success(false)
         add_error(Truemail::Validate::DomainListMatch::ERROR)
       end
@@ -19,7 +19,15 @@ module Truemail
       end
 
       def whitelisted_domain?
-        configuration.whitelisted_domains.include?(email_domain)
+        @whitelisted_domain ||= configuration.whitelisted_domains.include?(email_domain)
+      end
+
+      def whitelist_validation?
+        configuration.whitelist_validation
+      end
+
+      def whitelist_validation_case?
+        whitelist_validation? && !whitelisted_domain?
       end
 
       def blacklisted_domain?

data/lib/truemail/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: truemail
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.2.0
 platform: ruby
 authors:
 - Vladislav Trotsenko
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2019-06-18 00:00:00.000000000 Z
+date: 2019-06-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler