Merge pull request #11110 from annando/issue-11109
Issue 11109: Fix server detection
This commit is contained in:
commit
ff80e0876e
@ -349,9 +349,9 @@ class GServer
|
|||||||
}
|
}
|
||||||
|
|
||||||
// On a redirect follow the new host but mark the old one as failure
|
// On a redirect follow the new host but mark the old one as failure
|
||||||
if ($curlResult->isSuccess() && (parse_url($url, PHP_URL_HOST) != parse_url($curlResult->getRedirectUrl(), PHP_URL_HOST))) {
|
if ($curlResult->isSuccess() && !empty($curlResult->getRedirectUrl()) && (parse_url($url, PHP_URL_HOST) != parse_url($curlResult->getRedirectUrl(), PHP_URL_HOST))) {
|
||||||
$curlResult = DI::httpClient()->get($url, [HttpClientOptions::TIMEOUT => $xrd_timeout]);
|
$curlResult = DI::httpClient()->get($url, [HttpClientOptions::TIMEOUT => $xrd_timeout]);
|
||||||
if (parse_url($url, PHP_URL_HOST) != parse_url($curlResult->getRedirectUrl(), PHP_URL_HOST)) {
|
if (!empty($curlResult->getRedirectUrl()) && parse_url($url, PHP_URL_HOST) != parse_url($curlResult->getRedirectUrl(), PHP_URL_HOST)) {
|
||||||
Logger::info('Found redirect. Mark old entry as failure', ['old' => $url, 'new' => $curlResult->getRedirectUrl()]);
|
Logger::info('Found redirect. Mark old entry as failure', ['old' => $url, 'new' => $curlResult->getRedirectUrl()]);
|
||||||
self::setFailure($url);
|
self::setFailure($url);
|
||||||
self::detect($curlResult->getRedirectUrl(), $network, $only_nodeinfo);
|
self::detect($curlResult->getRedirectUrl(), $network, $only_nodeinfo);
|
||||||
@ -388,7 +388,7 @@ class GServer
|
|||||||
|
|
||||||
$curlResult = DI::httpClient()->get($baseurl, [HttpClientOptions::TIMEOUT => $xrd_timeout]);
|
$curlResult = DI::httpClient()->get($baseurl, [HttpClientOptions::TIMEOUT => $xrd_timeout]);
|
||||||
if ($curlResult->isSuccess()) {
|
if ($curlResult->isSuccess()) {
|
||||||
if ((parse_url($baseurl, PHP_URL_HOST) != parse_url($curlResult->getRedirectUrl(), PHP_URL_HOST))) {
|
if (!empty($curlResult->getRedirectUrl()) && (parse_url($baseurl, PHP_URL_HOST) != parse_url($curlResult->getRedirectUrl(), PHP_URL_HOST))) {
|
||||||
Logger::info('Found redirect. Mark old entry as failure', ['old' => $url, 'new' => $curlResult->getRedirectUrl()]);
|
Logger::info('Found redirect. Mark old entry as failure', ['old' => $url, 'new' => $curlResult->getRedirectUrl()]);
|
||||||
self::setFailure($url);
|
self::setFailure($url);
|
||||||
self::detect($curlResult->getRedirectUrl(), $network, $only_nodeinfo);
|
self::detect($curlResult->getRedirectUrl(), $network, $only_nodeinfo);
|
||||||
|
Loading…
x
Reference in New Issue
Block a user