Fixed: Log files should not be cached

src/Lidarr.Http/Frontend/Mappers/LogFileMapper.cs

@@ -6,11 +6,11 @@ using NzbDrone.Common.Extensions;
 
 namespace Lidarr.Http.Frontend.Mappers
 {
-    public class UpdateLogFileMapper : StaticResourceMapperBase
+    public class LogFileMapper : StaticResourceMapperBase
     {
         private readonly IAppFolderInfo _appFolderInfo;
 
-        public UpdateLogFileMapper(IAppFolderInfo appFolderInfo, IDiskProvider diskProvider, Logger logger)
+        public LogFileMapper(IAppFolderInfo appFolderInfo, IDiskProvider diskProvider, Logger logger)
             : base(diskProvider, logger)
         {
             _appFolderInfo = appFolderInfo;
@@ -21,12 +21,12 @@ namespace Lidarr.Http.Frontend.Mappers
             var path = resourceUrl.Replace('/', Path.DirectorySeparatorChar);
             path = Path.GetFileName(path);
 
-            return Path.Combine(_appFolderInfo.GetUpdateLogFolder(), path);
+            return Path.Combine(_appFolderInfo.GetLogFolder(), path);
         }
 
         public override bool CanHandle(string resourceUrl)
         {
-            return resourceUrl.StartsWith("/updatelogfile/") && resourceUrl.EndsWith(".txt");
+            return resourceUrl.StartsWith("/logfile/") && resourceUrl.EndsWith(".txt");
         }
     }
 }

src/Lidarr.Http/Frontend/Mappers/UpdateLogFileMapper.cs

@@ -6,11 +6,11 @@ using NzbDrone.Common.Extensions;
 
 namespace Lidarr.Http.Frontend.Mappers
 {
-    public class LogFileMapper : StaticResourceMapperBase
+    public class UpdateLogFileMapper : StaticResourceMapperBase
     {
         private readonly IAppFolderInfo _appFolderInfo;
 
-        public LogFileMapper(IAppFolderInfo appFolderInfo, IDiskProvider diskProvider, Logger logger)
+        public UpdateLogFileMapper(IAppFolderInfo appFolderInfo, IDiskProvider diskProvider, Logger logger)
             : base(diskProvider, logger)
         {
             _appFolderInfo = appFolderInfo;
@@ -21,12 +21,12 @@ namespace Lidarr.Http.Frontend.Mappers
             var path = resourceUrl.Replace('/', Path.DirectorySeparatorChar);
             path = Path.GetFileName(path);
 
-            return Path.Combine(_appFolderInfo.GetLogFolder(), path);
+            return Path.Combine(_appFolderInfo.GetUpdateLogFolder(), path);
         }
 
         public override bool CanHandle(string resourceUrl)
         {
-            return resourceUrl.StartsWith("/logfile/") && resourceUrl.EndsWith(".txt");
+            return resourceUrl.StartsWith("/updatelogfile/") && resourceUrl.EndsWith(".txt");
         }
     }
 }

src/Lidarr.Http/Middleware/CacheableSpecification.cs

@@ -39,23 +39,26 @@ namespace Lidarr.Http.Middleware
                 return false;
             }
 
-            if (request.Path.Value?.EndsWith("/index.js") ?? false)
+            var path = request.Path.Value ?? "";
+
+            if (path.EndsWith("/index.js"))
             {
                 return false;
             }
 
-            if (request.Path.Value?.EndsWith("/initialize.js") ?? false)
+            if (path.EndsWith("/initialize.js"))
             {
                 return false;
             }
 
-            if (request.Path.StartsWithSegments("/feed", StringComparison.CurrentCultureIgnoreCase))
+            if (path.StartsWith("/feed", StringComparison.CurrentCultureIgnoreCase))
             {
                 return false;
             }
 
-            if (request.Path.StartsWithSegments("/log", StringComparison.CurrentCultureIgnoreCase) &&
+            if ((path.StartsWith("/logfile", StringComparison.CurrentCultureIgnoreCase) ||
-                (request.Path.Value?.EndsWith(".txt", StringComparison.CurrentCultureIgnoreCase) ?? false))
+                path.StartsWith("/updatelogfile", StringComparison.CurrentCultureIgnoreCase)) &&
+                path.EndsWith(".txt", StringComparison.CurrentCultureIgnoreCase))
             {
                 return false;
             }