about summary refs log tree commit diff
path: root/app/validators/email_mx_validator.rb
diff options
context:
space:
mode:
authorClaire <claire.github-309c@sitedethib.com>2022-05-01 18:24:42 +0200
committerGitHub <noreply@github.com>2022-05-01 18:24:42 +0200
commita41b3d0457451abf750ecd2e5576e4d516d8ab95 (patch)
tree9e2238c87c72ef0bdca8159e4beb886553cd8755 /app/validators/email_mx_validator.rb
parent252deefe3433d0cedafd973becd0d85b5182eb49 (diff)
parent26a51291c74e7f27bf5e97bec9cd415d7ed48246 (diff)
Merge pull request #1760 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.rb6
1 files changed, 3 insertions, 3 deletions
diff --git a/app/validators/email_mx_validator.rb b/app/validators/email_mx_validator.rb
index 237ca4c7b..20f2fd37c 100644
--- a/app/validators/email_mx_validator.rb
+++ b/app/validators/email_mx_validator.rb
@@ -15,7 +15,7 @@ class EmailMxValidator < ActiveModel::Validator
 
       if resolved_ips.empty?
         user.errors.add(:email, :unreachable)
-      elsif on_blacklist?(resolved_domains, resolved_ips, user.sign_up_ip)
+      elsif on_blacklist?(resolved_domains, user.sign_up_ip)
         user.errors.add(:email, :blocked)
       end
     end
@@ -57,7 +57,7 @@ class EmailMxValidator < ActiveModel::Validator
     [ips, records]
   end
 
-  def on_blacklist?(domains, resolved_ips, attempt_ip)
-    EmailDomainBlock.block?(domains, ips: resolved_ips, attempt_ip: attempt_ip)
+  def on_blacklist?(domains, attempt_ip)
+    EmailDomainBlock.block?(domains, attempt_ip: attempt_ip)
   end
 end