diff options
| author | Claire <claire.github-309c@sitedethib.com> | 2021-03-25 00:31:15 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-03-25 00:31:15 +0100 |
| commit | a2a85d5ae03282c6f9cbf452b9b8f7d948a9f380 (patch) | |
| tree | f3f15ea30de78e0fabaa5135a3b3c7b51053e4b6 /app/validators/email_mx_validator.rb | |
| parent | d7c1c41859549212a6d34ad869fded16acc17b48 (diff) | |
| parent | 5ea53b6158c42660aa61da2a49a3bbf9e2e406d0 (diff) | |
Merge pull request #1516 from ClearlyClaire/glitch-soc/merge-upstream
Merge upstream changes
Diffstat (limited to 'app/validators/email_mx_validator.rb')
| -rw-r--r-- | app/validators/email_mx_validator.rb | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/app/validators/email_mx_validator.rb b/app/validators/email_mx_validator.rb index 9f70a1469..dceef5029 100644 --- a/app/validators/email_mx_validator.rb +++ b/app/validators/email_mx_validator.rb @@ -10,7 +10,7 @@ class EmailMxValidator < ActiveModel::Validator if domain.blank? user.errors.add(:email, :invalid) - else + elsif !on_allowlist?(domain) ips, hostnames = resolve_mx(domain) if ips.empty? @@ -33,6 +33,12 @@ class EmailMxValidator < ActiveModel::Validator nil end + def on_allowlist?(domain) + return false if Rails.configuration.x.email_domains_whitelist.blank? + + Rails.configuration.x.email_domains_whitelist.include?(domain) + end + def resolve_mx(domain) hostnames = [] ips = [] |