Fixed: (Flaresolverr) Proxy Test

Fixes #651
pull/653/head
Qstick 3 years ago
parent 5f5df99dab
commit 01e7e924c4

@ -149,7 +149,7 @@ namespace NzbDrone.Core.IndexerProxies.FlareSolverr
{
var failures = new List<ValidationFailure>();
var request = PreRequest(_cloudRequestBuilder.Create()
var request = GenerateFlareSolverrRequest(_cloudRequestBuilder.Create()
.Resource("/ping")
.Build());
@ -157,12 +157,13 @@ namespace NzbDrone.Core.IndexerProxies.FlareSolverr
{
var response = _httpClient.Execute(request);
// We only care about 400 responses, other error codes can be ignored
if (response.StatusCode == HttpStatusCode.BadRequest)
if (response.StatusCode != HttpStatusCode.OK)
{
_logger.Error("Proxy Health Check failed: {0}", response.StatusCode);
failures.Add(new NzbDroneValidationFailure("Host", string.Format(_localizationService.GetLocalizedString("ProxyCheckBadRequestMessage"), response.StatusCode)));
}
var result = JsonConvert.DeserializeObject<FlareSolverrResponse>(response.Content);
}
catch (Exception ex)
{

@ -31,12 +31,6 @@ namespace NzbDrone.Core.IndexerProxies
{
var failures = new List<ValidationFailure>();
var addresses = Dns.GetHostAddresses(Settings.Host);
if (!addresses.Any())
{
failures.Add(new NzbDroneValidationFailure("Host", string.Format(_localizationService.GetLocalizedString("ProxyCheckResolveIpMessage"), addresses)));
}
var request = PreRequest(_cloudRequestBuilder.Create()
.Resource("/ping")
.Build());
@ -55,7 +49,7 @@ namespace NzbDrone.Core.IndexerProxies
catch (Exception ex)
{
_logger.Error(ex, "Proxy Health Check failed");
failures.Add(new NzbDroneValidationFailure("Host", string.Format("Failed to test proxy: {0}", request.Url)));
failures.Add(new NzbDroneValidationFailure("Host", string.Format("Failed to test proxy: {0}", ex.Message)));
}
return new ValidationResult(failures);

Loading…
Cancel
Save