diff options
| author | Côme Chilliet <come.chilliet@nextcloud.com> | 2022-06-28 11:35:45 +0200 |
|---|---|---|
| committer | Côme Chilliet (Rebase PR Action) <come-nc@users.noreply.github.com> | 2022-07-12 09:49:27 +0000 |
| commit | d0830432a7ce4dece2af7a5ca3c0f3831dbd1291 (patch) | |
| tree | f07a147e487ea9221948d08cb38c8e602a0a3d61 /lib/private | |
| parent | bd9aff47b69d62b42dd3e450ea76bb1616dbec58 (diff) | |
| download | nextcloud-server-d0830432a7ce4dece2af7a5ca3c0f3831dbd1291.tar.gz nextcloud-server-d0830432a7ce4dece2af7a5ca3c0f3831dbd1291.zip | |
Refactor local IP if and set strict to true for in_array
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Diffstat (limited to 'lib/private')
| -rw-r--r-- | lib/private/Http/Client/LocalAddressChecker.php | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/lib/private/Http/Client/LocalAddressChecker.php b/lib/private/Http/Client/LocalAddressChecker.php index b233f34b19c..749a1a97c8a 100644 --- a/lib/private/Http/Client/LocalAddressChecker.php +++ b/lib/private/Http/Client/LocalAddressChecker.php @@ -36,13 +36,13 @@ class LocalAddressChecker { } public function ThrowIfLocalIp(string $ip) : void { - if ((bool)filter_var($ip, FILTER_VALIDATE_IP) && !filter_var($ip, FILTER_VALIDATE_IP, FILTER_FLAG_NO_PRIV_RANGE | FILTER_FLAG_NO_RES_RANGE)) { - $this->logger->warning("Host $ip was not connected to because it violates local access rules"); - throw new LocalServerException('Host violates local access rules'); - } - $localIps = ['100.100.100.200']; - if ((bool)filter_var($ip, FILTER_VALIDATE_IP) && in_array($ip, $localIps)) { + if ( + (bool)filter_var($ip, FILTER_VALIDATE_IP) && + ( + !filter_var($ip, FILTER_VALIDATE_IP, FILTER_FLAG_NO_PRIV_RANGE | FILTER_FLAG_NO_RES_RANGE) || + in_array($ip, $localIps, true) + )) { $this->logger->warning("Host $ip was not connected to because it violates local access rules"); throw new LocalServerException('Host violates local access rules'); } |