mirror of
https://github.com/FreshRSS/FreshRSS.git
synced 2026-10-28 14:08:45 -04:00
* Fix local DNS leak when fetching feeds through a proxy Follow-up to #9390: `getRetryAfterFile()` passed the full URL to `Minz_Request::serverIsPublic()`, which then resolved the target hostname with the local resolver (`dns_get_record()`), even when a proxy was configured. With a socks5h/socks4a proxy this leaks the target hostname to the local DNS and defeats remote name resolution. - `Minz_Request::serverIsPublic()`: add an `$allowDnsResolve` parameter, and forward it to the recursive call so the guard holds for the whole recursion. - `FreshRSS_http_Util::getRetryAfterFile()`: do not resolve the host name when a proxy is configured (`$proxy === ''`), since the proxy is responsible for DNS. --------- Co-authored-by: Inverle <inverle@proton.me>