diff options
| author | Cody Robibero <cody@robibe.ro> | 2024-10-06 16:20:14 -0600 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-10-06 16:20:14 -0600 |
| commit | 6a9a9546fd9050e29339d0af605f26d88bdca3e0 (patch) | |
| tree | a6a14f03d3e1304798fba62d8fa97474259f4312 | |
| parent | b988b989d5b2c58f245eaf5f752ed643d6f04258 (diff) | |
Clean up logging in MediaSegmentManager (#12776)
| -rw-r--r-- | Jellyfin.Server.Implementations/MediaSegments/MediaSegmentManager.cs | 19 |
1 files changed, 14 insertions, 5 deletions
diff --git a/Jellyfin.Server.Implementations/MediaSegments/MediaSegmentManager.cs b/Jellyfin.Server.Implementations/MediaSegments/MediaSegmentManager.cs index 9953c05be..d641f521b 100644 --- a/Jellyfin.Server.Implementations/MediaSegments/MediaSegmentManager.cs +++ b/Jellyfin.Server.Implementations/MediaSegments/MediaSegmentManager.cs @@ -66,16 +66,21 @@ public class MediaSegmentManager : IMediaSegmentManager }) .ToList(); - _logger.LogInformation("Start media segment extraction from providers with {CountProviders} enabled", providers.Count); + if (providers.Count == 0) + { + _logger.LogDebug("Skipping media segment extraction as no providers are enabled for {MediaPath}", baseItem.Path); + return; + } + using var db = await _dbProvider.CreateDbContextAsync(cancellationToken).ConfigureAwait(false); if (!overwrite && (await db.MediaSegments.AnyAsync(e => e.ItemId.Equals(baseItem.Id), cancellationToken).ConfigureAwait(false))) { - _logger.LogInformation("Skip {MediaPath} as it already contains media segments", baseItem.Path); + _logger.LogDebug("Skip {MediaPath} as it already contains media segments", baseItem.Path); return; } - _logger.LogInformation("Clear existing Segments for {MediaPath}", baseItem.Path); + _logger.LogDebug("Start media segment extraction for {MediaPath} with {CountProviders} providers enabled", baseItem.Path, providers.Count); await db.MediaSegments.Where(e => e.ItemId.Equals(baseItem.Id)).ExecuteDeleteAsync(cancellationToken).ConfigureAwait(false); @@ -86,15 +91,19 @@ public class MediaSegmentManager : IMediaSegmentManager { if (!await provider.Supports(baseItem).ConfigureAwait(false)) { - _logger.LogDebug("Media Segment provider {ProviderName} does not support item with path {Path}", provider.Name, baseItem.Path); + _logger.LogDebug("Media Segment provider {ProviderName} does not support item with path {MediaPath}", provider.Name, baseItem.Path); continue; } - _logger.LogDebug("Run Media Segment provider {ProviderName}", provider.Name); try { var segments = await provider.GetMediaSegments(requestItem, cancellationToken) .ConfigureAwait(false); + if (segments.Count == 0) + { + _logger.LogDebug("Media Segment provider {ProviderName} did not find any segments for {MediaPath}", provider.Name, baseItem.Path); + continue; + } _logger.LogInformation("Media Segment provider {ProviderName} found {CountSegments} for {MediaPath}", provider.Name, segments.Count, baseItem.Path); var providerId = GetProviderId(provider.Name); |
