DNS record check now passes if email address has no top-level domain (#…

…355)
egulias · Jan 14, 2023 · 3a85486 · 3a85486
1 parent db4c314
commit 3a85486
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 3 deletions.
diff --git a/src/Validation/DNSCheckValidation.php b/src/Validation/DNSCheckValidation.php
@@ -123,9 +123,20 @@ protected function checkDns($host)
  {
  $variant = INTL_IDNA_VARIANT_UTS46;
 
- $host = rtrim(idn_to_ascii($host, IDNA_DEFAULT, $variant), '.') . '.';
+ $host = rtrim(idn_to_ascii($host, IDNA_DEFAULT, $variant), '.');
 
- return $this->validateDnsRecords($host);
+ $hostParts = explode('.', $host);
+ $host = array_pop($hostParts);
+
+ while (count($hostParts) > 0) {
+ $host = array_pop($hostParts) . '.' . $host;
+
+ if ($this->validateDnsRecords($host)) {
+ return true;
+ }
+ }
+
+ return false;
  }
 
 

diff --git a/tests/EmailValidator/Validation/DNSCheckValidationTest.php b/tests/EmailValidator/Validation/DNSCheckValidationTest.php
@@ -21,6 +21,7 @@ public function validEmailsProvider()
  return [
  // dot-atom
  ['[email protected]'],
+ ['[email protected]'],
  ['[email protected]'],
  ['[email protected]'],
  ['user+mailbox/[email protected]'],
@@ -149,4 +150,4 @@ public function getRecords(string $host, int $type): DNSRecords
  $validation->isValid('[email protected]', new EmailLexer());
  $this->assertEquals($expectedError, $validation->getError());
  }
-}
+}