From 16a8132bec7299e97581f221920dd9d966a43d7a Mon Sep 17 00:00:00 2001
From: Steve Boyd <emteknetnz@gmail.com>
Date: Tue, 20 Aug 2024 09:25:22 +1200
Subject: [PATCH] DEP Use symfony for IPUtils

---
 composer.json                                     |  1 +
 src/Control/Middleware/TrustedProxyMiddleware.php |  2 +-
 src/Control/Util/IPUtils.php                      |  7 +++++++
 tests/php/Control/IPUtilsTest.php                 | 13 ++++++++++---
 4 files changed, 19 insertions(+), 4 deletions(-)

diff --git a/composer.json b/composer.json
index b634895127d..80e8c658cd2 100644
--- a/composer.json
+++ b/composer.json
@@ -44,6 +44,7 @@
         "symfony/config": "^6.1",
         "symfony/dom-crawler": "^6.1",
         "symfony/filesystem": "^6.1",
+        "symfony/http-foundation": "^6.1",
         "symfony/mailer": "^6.1",
         "symfony/mime": "^6.1",
         "symfony/translation": "^6.1",
diff --git a/src/Control/Middleware/TrustedProxyMiddleware.php b/src/Control/Middleware/TrustedProxyMiddleware.php
index f9017bce4e2..e990646917c 100644
--- a/src/Control/Middleware/TrustedProxyMiddleware.php
+++ b/src/Control/Middleware/TrustedProxyMiddleware.php
@@ -3,7 +3,7 @@
 namespace SilverStripe\Control\Middleware;
 
 use SilverStripe\Control\HTTPRequest;
-use SilverStripe\Control\Util\IPUtils;
+use Symfony\Component\HttpFoundation\IpUtils;
 
 /**
  * This middleware will rewrite headers that provide IP and host details from an upstream proxy.
diff --git a/src/Control/Util/IPUtils.php b/src/Control/Util/IPUtils.php
index be3cdc52257..8d6039ad317 100644
--- a/src/Control/Util/IPUtils.php
+++ b/src/Control/Util/IPUtils.php
@@ -11,10 +11,13 @@
 
 namespace SilverStripe\Control\Util;
 
+use SilverStripe\Dev\Deprecation;
+
 /**
  * Http utility functions.
  *
  * @author Fabien Potencier <fabien@symfony.com>
+ * @deprecated 5.3.0 Use Symfony\Component\HttpFoundation\IpUtils instead
  */
 class IPUtils
 {
@@ -37,6 +40,7 @@ private function __construct()
      */
     public static function checkIP($requestIP, $ips)
     {
+        Deprecation::notice('5.3.0', 'Use Symfony\Component\HttpFoundation\IpUtils::checkIP() instead');
         if (!is_array($ips)) {
             $ips = [$ips];
         }
@@ -62,6 +66,7 @@ public static function checkIP($requestIP, $ips)
      */
     public static function checkIP4($requestIP, $ip)
     {
+        Deprecation::notice('5.3.0', 'Use Symfony\Component\HttpFoundation\IpUtils::checkIP4() instead');
         if (!filter_var($requestIP, FILTER_VALIDATE_IP, FILTER_FLAG_IPV4)) {
             return false;
         }
@@ -100,6 +105,7 @@ public static function checkIP4($requestIP, $ip)
      */
     public static function checkIP6($requestIP, $ip)
     {
+        Deprecation::notice('5.3.0', 'Use Symfony\Component\HttpFoundation\IpUtils::checkIP6() instead');
         if (!((extension_loaded('sockets') && defined('AF_INET6')) || @inet_pton('::1'))) {
             throw new \RuntimeException('Unable to check IPv6. Check that PHP was not compiled with option "disable-ipv6".');
         }
@@ -141,6 +147,7 @@ public static function checkIP6($requestIP, $ip)
      */
     public static function anonymize(string $ip): string
     {
+        Deprecation::notice('5.3.0', 'Use Symfony\Component\HttpFoundation\IpUtils::anonymize() instead');
         $wrappedIPv6 = false;
         if (str_starts_with($ip, '[') && str_ends_with($ip, ']')) {
             $wrappedIPv6 = true;
diff --git a/tests/php/Control/IPUtilsTest.php b/tests/php/Control/IPUtilsTest.php
index 4c4a20e0214..8b0ddfe4697 100644
--- a/tests/php/Control/IPUtilsTest.php
+++ b/tests/php/Control/IPUtilsTest.php
@@ -13,6 +13,7 @@
  */
 use SilverStripe\Dev\SapphireTest;
 use SilverStripe\Control\Util\IPUtils;
+use SilverStripe\Dev\Deprecation;
 
 class IPUtilsTest extends SapphireTest
 {
@@ -21,7 +22,9 @@ class IPUtilsTest extends SapphireTest
      */
     public function testIPv4($matches, $remoteAddr, $cidr)
     {
-        $this->assertSame($matches, IPUtils::checkIP($remoteAddr, $cidr));
+        Deprecation::withNoReplacement(function () use ($matches, $remoteAddr, $cidr) {
+            $this->assertSame($matches, IPUtils::checkIP($remoteAddr, $cidr));
+        });
     }
 
     public function iPv4Provider()
@@ -51,7 +54,9 @@ public function testIPv6($matches, $remoteAddr, $cidr)
             $this->markTestSkipped('Only works when PHP is compiled without the option "disable-ipv6".');
         }
 
-        $this->assertSame($matches, IPUtils::checkIP($remoteAddr, $cidr));
+        Deprecation::withNoReplacement(function () use ($matches, $remoteAddr, $cidr) {
+            $this->assertSame($matches, IPUtils::checkIP($remoteAddr, $cidr));
+        });
     }
 
     public function iPv6Provider()
@@ -80,6 +85,8 @@ public function testAnIPv6WithOptionDisabledIPv6()
             $this->markTestSkipped('Only works when PHP is compiled with the option "disable-ipv6".');
         }
 
-        IPUtils::checkIP('2a01:198:603:0:396e:4789:8e99:890f', '2a01:198:603:0::/65');
+        Deprecation::withNoReplacement(function () {
+            IPUtils::checkIP('2a01:198:603:0:396e:4789:8e99:890f', '2a01:198:603:0::/65');
+        });
     }
 }