summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMorris Jobke <hey@morrisjobke.de>2019-03-06 10:34:02 +0100
committerGitHub <noreply@github.com>2019-03-06 10:34:02 +0100
commitd004164fcc4dc887cae388e68f8ded3b4ea166ad (patch)
treeeb1187ef647dc25f4908b6dfa77268e174813636
parent58f6a777b1cad989cca01bda029d0a71e3c7d16c (diff)
parentc9b588774bf57b0d0f9742b2ca32f2185c4317f1 (diff)
downloadnextcloud-server-d004164fcc4dc887cae388e68f8ded3b4ea166ad.tar.gz
nextcloud-server-d004164fcc4dc887cae388e68f8ded3b4ea166ad.zip
Merge pull request #13327 from nextcloud/allow-bracket-notation-for-remove-ipv6-address
Allow bracket IPv6 address format inside IPAdress Normalizer
-rw-r--r--lib/private/Security/Normalizer/IpAddress.php3
-rw-r--r--tests/lib/Security/Normalizer/IpAddressTest.php4
2 files changed, 7 insertions, 0 deletions
diff --git a/lib/private/Security/Normalizer/IpAddress.php b/lib/private/Security/Normalizer/IpAddress.php
index 84c87054c76..4edf83eca5e 100644
--- a/lib/private/Security/Normalizer/IpAddress.php
+++ b/lib/private/Security/Normalizer/IpAddress.php
@@ -68,6 +68,9 @@ class IpAddress {
* @return string
*/
private function getIPv6Subnet(string $ip, int $maskBits = 48): string {
+ if ($ip[0] === '[' && $ip[-1] === ']') { // If IP is with brackets, for example [::1]
+ $ip = substr($ip, 1, strlen($ip) - 2);
+ }
$binary = \inet_pton($ip);
for ($i = 128; $i > $maskBits; $i -= 8) {
$j = \intdiv($i, 8) - 1;
diff --git a/tests/lib/Security/Normalizer/IpAddressTest.php b/tests/lib/Security/Normalizer/IpAddressTest.php
index 36a48f601d3..97a8737ea9e 100644
--- a/tests/lib/Security/Normalizer/IpAddressTest.php
+++ b/tests/lib/Security/Normalizer/IpAddressTest.php
@@ -40,6 +40,10 @@ class IpAddressTest extends TestCase {
'2001:0db8:85a3:0000:0000:8a2e:0370:7334',
'2001:db8:85a3::8a2e:370:7334/128',
],
+ [
+ '[::1]',
+ '::1/128',
+ ],
];
}