diff --git a/allstarr/Controllers/AdminController.cs b/allstarr/Controllers/AdminController.cs index b75571d..164e2d8 100644 --- a/allstarr/Controllers/AdminController.cs +++ b/allstarr/Controllers/AdminController.cs @@ -537,18 +537,18 @@ public class AdminController : ControllerBase { try { - using var doc = JsonDocument.Parse(externalMappingJson); - var root = doc.RootElement; + using var extDoc = JsonDocument.Parse(externalMappingJson); + var extRoot = extDoc.RootElement; string? provider = null; string? externalId = null; - if (root.TryGetProperty("provider", out var providerEl)) + if (extRoot.TryGetProperty("provider", out var providerEl)) { provider = providerEl.GetString(); } - if (root.TryGetProperty("id", out var idEl)) + if (extRoot.TryGetProperty("id", out var idEl)) { externalId = idEl.GetString(); } @@ -674,12 +674,12 @@ public class AdminController : ControllerBase { try { - using var doc = JsonDocument.Parse(externalMappingJson); - var root = doc.RootElement; + using var extDoc = JsonDocument.Parse(externalMappingJson); + var extRoot = extDoc.RootElement; string? provider = null; - if (root.TryGetProperty("provider", out var providerEl)) + if (extRoot.TryGetProperty("provider", out var providerEl)) { provider = providerEl.GetString(); } diff --git a/allstarr/Services/Common/RedisCacheService.cs b/allstarr/Services/Common/RedisCacheService.cs index 16c938c..063f8c5 100644 --- a/allstarr/Services/Common/RedisCacheService.cs +++ b/allstarr/Services/Common/RedisCacheService.cs @@ -58,27 +58,13 @@ public class RedisCacheService { var value = await _db!.StringGetAsync(key); - // Only log manual/external mapping HITs (not MISSes - they're expected) - var isManualMapping = key.Contains(":manual-map:") || key.Contains(":external-map:"); - if (value.HasValue) { - if (isManualMapping) - { - _logger.LogInformation("Redis cache HIT: {Key}", key); - } - else - { - _logger.LogDebug("Redis cache HIT: {Key}", key); - } + _logger.LogDebug("Redis cache HIT: {Key}", key); } else { - // Don't log MISS for manual/external mappings - they're expected to be missing most of the time - if (!isManualMapping) - { - _logger.LogDebug("Redis cache MISS: {Key}", key); - } + _logger.LogDebug("Redis cache MISS: {Key}", key); } return value; } @@ -120,16 +106,7 @@ public class RedisCacheService var result = await _db!.StringSetAsync(key, value, expiry); if (result) { - // Log manual/external mappings at INFO level, others at DEBUG - var isManualMapping = key.Contains(":manual-map:") || key.Contains(":external-map:"); - if (isManualMapping) - { - _logger.LogInformation("Redis cache SET: {Key} (TTL: {Expiry})", key, expiry?.ToString() ?? "none"); - } - else - { - _logger.LogDebug("Redis cache SET: {Key} (TTL: {Expiry})", key, expiry?.ToString() ?? "none"); - } + _logger.LogDebug("Redis cache SET: {Key} (TTL: {Expiry})", key, expiry?.ToString() ?? "none"); } return result; }