Improve Release Grabbing & Failure Logging

(cherry picked from commit d7aea82e45a7c5fec9e72b534fc4c9fb8654c519)

Closes #4534
This commit is contained in:
bakerboy448 2024-01-27 11:56:08 -06:00 committed by Bogdan
parent ef8aeda3b5
commit dec241e6b6
1 changed files with 5 additions and 6 deletions

View File

@ -195,31 +195,30 @@ namespace NzbDrone.Core.Download
private async Task<ProcessedDecisionResult> ProcessDecisionInternal(DownloadDecision decision, int? downloadClientId = null) private async Task<ProcessedDecisionResult> ProcessDecisionInternal(DownloadDecision decision, int? downloadClientId = null)
{ {
var remoteAlbum = decision.RemoteAlbum; var remoteAlbum = decision.RemoteAlbum;
var remoteIndexer = remoteAlbum.Release.Indexer;
try try
{ {
_logger.Trace("Grabbing from Indexer {0} at priority {1}.", remoteAlbum.Release.Indexer, remoteAlbum.Release.IndexerPriority); _logger.Trace("Grabbing release '{0}' from Indexer {1} at priority {2}.", remoteAlbum, remoteIndexer, remoteAlbum.Release.IndexerPriority);
await _downloadService.DownloadReport(remoteAlbum, downloadClientId); await _downloadService.DownloadReport(remoteAlbum, downloadClientId);
return ProcessedDecisionResult.Grabbed; return ProcessedDecisionResult.Grabbed;
} }
catch (ReleaseUnavailableException) catch (ReleaseUnavailableException)
{ {
_logger.Warn("Failed to download release from indexer, no longer available. " + remoteAlbum); _logger.Warn("Failed to download release '{0}' from Indexer {1}. Release not available", remoteAlbum, remoteIndexer);
return ProcessedDecisionResult.Rejected; return ProcessedDecisionResult.Rejected;
} }
catch (Exception ex) catch (Exception ex)
{ {
if (ex is DownloadClientUnavailableException || ex is DownloadClientAuthenticationException) if (ex is DownloadClientUnavailableException || ex is DownloadClientAuthenticationException)
{ {
_logger.Debug(ex, _logger.Debug(ex, "Failed to send release '{0}' from Indexer {1} to download client, storing until later.", remoteAlbum, remoteIndexer);
"Failed to send release to download client, storing until later. " + remoteAlbum);
return ProcessedDecisionResult.Failed; return ProcessedDecisionResult.Failed;
} }
else else
{ {
_logger.Warn(ex, "Couldn't add report to download queue. " + remoteAlbum); _logger.Warn(ex, "Couldn't add release '{0}' from Indexer {1} to download queue.", remoteAlbum, remoteIndexer);
return ProcessedDecisionResult.Skipped; return ProcessedDecisionResult.Skipped;
} }
} }