Remove dependency on ext-intl

Fixes #6

Author: TRowbotham

CHANGELOG.md

@@ -5,6 +5,7 @@
 ### Added
 
 - Added test coverage on Windows 32-bit.
+- Dependency on `rowbot/idna` to support international domain names without `ext-intl`.
 
 ### Changed
 
@@ -18,6 +19,10 @@
 - Incorrect output encoding used when encoding was overriden to be "replacement", "utf-16", "utf-16le", or "utf-16be".
 - `ext-json` is now correctly listed as a dependency.
 
+### Removed
+
+- Dependency on `ext-intl` and `lib-ICU`.
+
 ## [3.0.1] - 2020-02-29
 
 ### Added

composer.json

@@ -6,17 +6,16 @@
     "authors": [
         {
             "name": "Trevor Rowbotham",
-            "homepage": "http://trowbotham.com",
+            "homepage": "https://trowbotham.com",
             "role": "Developer"
         }
     ],
     "require": {
         "php": ">=7.1",
-        "ext-intl": "*",
         "ext-json": "*",
         "ext-mbstring": "*",
-        "lib-ICU": ">=4.6",
-        "brick/math": "^0.8.13"
+        "brick/math": "^0.8.13",
+        "rowbot/idna": "^0.1.2"
     },
     "require-dev": {
         "guzzlehttp/guzzle": "^6.3",

phpstan.neon

@@ -14,7 +14,7 @@ parameters:
             path: src/URLSearchParams.php
 
         -
-            message: '#Strict comparison using !== between 0 and 0 will always evaluate to false\.#'
+            message: '#Strict comparison using === between string and false will always evaluate to false\.#'
             path: src/String/AbstractUSVString.php
 
         -
@@ -29,7 +29,3 @@ parameters:
         -
             message: '#Variable property access on Rowbot\\URL\\URLRecord\.#'
             path: src/State/AuthorityState.php
-
-        -
-            message: '#Property Rowbot\\URL\\String\\Utf8StringIterator::\$codePoints \(array<int, string>\) does not accept array<string, mixed>\.#'
-            path: src/String/Utf8StringIterator.php

src/Component/Host/HostParser.php

@@ -4,6 +4,7 @@
 
 namespace Rowbot\URL\Component\Host;
 
+use Rowbot\Idna\Idna;
 use Rowbot\URL\Component\Host\Exception\IDNATransformException;
 use Rowbot\URL\Component\Host\Exception\InvalidHostException;
 use Rowbot\URL\Component\Host\Exception\InvalidIPv4AddressException;
@@ -28,27 +29,32 @@ class HostParser
     private const FORBIDDEN_OPAQUE_HOST_CODEPOINTS = '\x00\x09\x0A\x0D\x20#\/:<>?@[\\\\\]^';
     private const FORBIDDEN_HOST_CODEPOINTS = self::FORBIDDEN_OPAQUE_HOST_CODEPOINTS . '%';
 
+    /**
+     * @see https://url.spec.whatwg.org/#concept-domain-to-ascii
+     */
     private function domainToAscii(string $domain, bool $beStrict = false): StringHost
     {
-        $flags = IDNA::CHECK_BIDI | IDNA::CHECK_JOINERS | IDNA::NONTRANSITIONAL_PROCESSING;
-
-        if ($beStrict) {
-            $flags |= IDNA::USE_STD3_ASCII_RULES | IDNA::VERIFY_DNS_LENGTH;
+        $result = Idna::toAscii($domain, [
+            'CheckHyphens'            => false,
+            'CheckBidi'               => true,
+            'CheckJoiners'            => true,
+            'UseSTD3ASCIIRules'       => $beStrict,
+            'Transitional_Processing' => false,
+            'VerifyDnsLength'         => $beStrict,
+        ]);
+        $convertedDomain = $result->getDomain();
+
+        if ($convertedDomain === '') {
+            // Validation error.
+            throw new IDNATransformException();
         }
 
-        try {
-            $result = IDNA::toAscii($domain, $flags);
-
-            if ($result === '') {
-                // Validation error.
-                throw new IDNATransformException();
-            }
-
-            return new StringHost($result);
-        } catch (IDNATransformException $e) {
+        if ($result->hasErrors()) {
             // Validation error.
-            throw $e;
+            throw new IDNATransformException();
         }
+
+        return new StringHost($convertedDomain);
     }
 
     /**