Просмотр исходного кода

Merge pull request #615 from mhujer/gh-612

Zend_Validate_Hostname allows "0" subdomain
Frank Brückner 10 лет назад
Родитель
Сommit
f61d12c000
2 измененных файлов с 11 добавлено и 1 удалено
  1. 1 1
      library/Zend/Validate/Hostname.php
  2. 10 0
      tests/Zend/Validate/HostnameTest.php

+ 1 - 1
library/Zend/Validate/Hostname.php

@@ -1459,7 +1459,7 @@ class Zend_Validate_Hostname extends Zend_Validate_Abstract
                     $check = 0;
                     foreach ($domainParts as $domainPart) {
                         // If some domain part is empty (i.e. zend..com), it's invalid
-                        if (empty($domainPart)) {
+                        if (empty($domainPart) && $domainPart !== '0') {
                             $this->_error(self::INVALID_HOSTNAME);
                             return false;
                         }

+ 10 - 0
tests/Zend/Validate/HostnameTest.php

@@ -536,4 +536,14 @@ class Zend_Validate_HostnameTest extends PHPUnit_Framework_TestCase
         $validator = new Zend_Validate_Hostname();
         $this->assertTrue($validator->isValid('straße.de'));
     }
+
+    /**
+     * @group GH-612
+     */
+    public function testZeroSubdomain()
+    {
+        $validator = new Zend_Validate_Hostname();
+        $this->assertTrue($validator->isValid('1.pool.ntp.org'));
+        $this->assertTrue($validator->isValid('0.pool.ntp.org'));
+    }
 }