Fix `http_code` usage in case of failures

This commit is contained in:
Philipp Holzer 2021-08-29 14:05:38 +02:00
parent 58001c729f
commit 0d6884a8a9
No known key found for this signature in database
GPG Key ID: 9A28B7D4FF5667BD
2 changed files with 3 additions and 6 deletions

View File

@ -146,11 +146,11 @@ class HTTPClient implements IHTTPClient
$exception->hasResponse()) {
return new GuzzleResponse($exception->getResponse(), $url, $exception->getCode(), '');
} else {
return new CurlResult($url, '', ['http_code' => $exception->getCode()], $exception->getCode(), '');
return new CurlResult($url, '', ['http_code' => 500], $exception->getCode(), '');
}
} catch (InvalidArgumentException | \InvalidArgumentException $argumentException) {
$this->logger->info('Invalid Argument for HTTP call.', ['url' => $url, 'method' => $method, 'exception' => $argumentException]);
return new CurlResult($url, '', ['http_code' => $argumentException->getCode()], $argumentException->getCode(), $argumentException->getMessage());
return new CurlResult($url, '', ['http_code' => 500], $argumentException->getCode(), $argumentException->getMessage());
} finally {
$this->logger->debug('Request stop.', ['url' => $url, 'method' => $method]);
$this->profiler->stopRecording();

View File

@ -28,9 +28,6 @@ class HTTPClientTest extends MockedTest
new Response(301, ['Location' => 'https:///']),
]));
$httpClient = DI::httpClient();
$httpClient->get('https:///');
self::assertEquals(1,1);
self::assertFalse(DI::httpClient()->get('https://friendica.local')->isSuccess());
}
}