diff options
| author |  Alexandre Alapetite <alexandre@alapetite.fr>
| 2018-10-27 17:19:08 +0200 |
|---|---|---|
| committer |  GitHub <noreply@github.com>
| 2018-10-27 17:19:08 +0200 |
| commit | 1f4dc71d95623aa18f4ac248a2f3c763d79ab1a2 (patch) | |
| tree | ba13ac0b883ad8bc7caa3083086a325c7d27dac3 /lib | |
| parent | 9fa2fc2da78ebd5ad8b6d8f0ee46f82f43711ded (diff) | |
Fix public IP detection (#2084)
* Fix public IP detection
gethostbyname() may not return the expected public IP in a container
deployment or behind a proxy.
https://github.com/FreshRSS/FreshRSS/pull/2010/files#r228714764
Might have set to false `pubsubhubbub_enabled` some installations made
from stratch from version 1.11.2.
* Changelog 2084
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/lib_rss.php | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/lib/lib_rss.php b/lib/lib_rss.php index c816f946e..4087f6faf 100644 --- a/lib/lib_rss.php +++ b/lib/lib_rss.php @@ -125,9 +125,8 @@ function server_is_public($address) { )); if ($is_public) { - $ip = gethostbyname($host); - $is_public &= !preg_match('/^(10|127|172[.]16|192[.]168)[.]/', $ip); - $is_public &= !preg_match('/^(\[)?(::1$|fc00::|fe80::)/i', $ip); + $is_public &= !preg_match('/^(10|127|172[.]16|192[.]168)[.]/', $host); + $is_public &= !preg_match('/^(\[)?(::1$|fc00::|fe80::)/i', $host); } return (bool)$is_public; |