Skip to content

Commit

Permalink
Fixed: (HealthCheck) Check only enabled indexer proxies
Browse files Browse the repository at this point in the history
  • Loading branch information
mynameisbogdan committed May 28, 2023
1 parent 6897ed0 commit b02188a
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 6 deletions.
10 changes: 6 additions & 4 deletions src/NzbDrone.Core/HealthCheck/Checks/IndexerProxyCheck.cs
Expand Up @@ -22,16 +22,18 @@ public class IndexerProxyCheck : HealthCheckBase

public override HealthCheck Check()
{
var enabledProviders = _proxyFactory.GetAvailableProviders();
var enabledProxies = _proxyFactory.GetAvailableProviders()
.Where(n => ((IndexerProxyDefinition)n.Definition).Enable)
.ToList();

var badProxies = enabledProviders.Where(p => p.Test().IsValid == false).ToList();
var badProxies = enabledProxies.Where(p => p.Test().IsValid == false).ToList();

if (enabledProviders.Empty() || badProxies.Count == 0)
if (enabledProxies.Empty() || badProxies.Count == 0)
{
return new HealthCheck(GetType());
}

if (badProxies.Count == enabledProviders.Count)
if (badProxies.Count == enabledProxies.Count)
{
return new HealthCheck(GetType(),
HealthCheckResult.Error,
Expand Down
4 changes: 2 additions & 2 deletions src/NzbDrone.Core/IndexerProxies/HttpIndexerProxyBase.cs
Expand Up @@ -31,8 +31,8 @@ public override ValidationResult Test()
var failures = new List<ValidationFailure>();

var request = PreRequest(_cloudRequestBuilder.Create()
.Resource("/ping")
.Build());
.Resource("/ping")
.Build());

try
{
Expand Down

0 comments on commit b02188a

Please sign in to comment.