diff options
author | Eugen Rochko <eugen@zeonfederated.com> | 2018-05-27 04:58:08 +0200 |
---|---|---|
committer | Yamagishi Kazutoshi <ykzts@desire.sh> | 2018-05-27 11:58:08 +0900 |
commit | 63c7b9157274f57c496399a1a5c728b32415034c (patch) | |
tree | aa83c2dfb3a201eb304757e18a3aafdf5d1a88d8 /app/validators | |
parent | 182bdbc5f4f0194cdd7b771246304c2dfc6f19e0 (diff) |
Validate that e-mail resolves with MX and it's not blacklisted (#7631)
Original patch by @j-a4
Diffstat (limited to 'app/validators')
-rw-r--r-- | app/validators/email_mx_validator.rb | 25 |
1 files changed, 25 insertions, 0 deletions
diff --git a/app/validators/email_mx_validator.rb b/app/validators/email_mx_validator.rb new file mode 100644 index 000000000..d4c7cc252 --- /dev/null +++ b/app/validators/email_mx_validator.rb @@ -0,0 +1,25 @@ +# frozen_string_literal: true + +require 'resolv' + +class EmailMxValidator < ActiveModel::Validator + def validate(user) + return if Rails.env.test? + user.errors.add(:email, I18n.t('users.invalid_email')) if invalid_mx?(user.email) + end + + private + + def invalid_mx?(value) + _, domain = value.split('@', 2) + + return true if domain.nil? + + records = Resolv::DNS.new.getresources(domain, Resolv::DNS::Resource::IN::MX).to_a.map { |e| e.exchange.to_s } + records.empty? || on_blacklist?(records) + end + + def on_blacklist?(values) + EmailDomainBlock.where(domain: values).any? + end +end |