From 2b0da546c9dae40fbc1b2654387be80a17c1848f Mon Sep 17 00:00:00 2001 From: ta264 Date: Wed, 4 Aug 2021 21:42:41 +0100 Subject: [PATCH] Tidy conversion to aspnetcore (cherry picked from commit 490f6e2e6aa3f220cc98f257a3ca3b2bea48fb80) (cherry picked from commit 8f3f90d4078d9d072d8ad4ccc3be35963b7435d6) --- .../Authentication/AuthenticationController.cs | 5 ++--- src/Lidarr.Http/Frontend/Mappers/HtmlMapperBase.cs | 9 --------- src/Lidarr.Http/Middleware/CacheableSpecification.cs | 6 +++--- .../REST/Attributes/RestGetByIdAttribute.cs | 10 ++-------- 4 files changed, 7 insertions(+), 23 deletions(-) diff --git a/src/Lidarr.Http/Authentication/AuthenticationController.cs b/src/Lidarr.Http/Authentication/AuthenticationController.cs index a95be2689..f68d68a45 100644 --- a/src/Lidarr.Http/Authentication/AuthenticationController.cs +++ b/src/Lidarr.Http/Authentication/AuthenticationController.cs @@ -14,12 +14,10 @@ namespace Lidarr.Http.Authentication public class AuthenticationController : Controller { private readonly IAuthenticationService _authService; - private readonly IConfigFileProvider _configFileProvider; - public AuthenticationController(IAuthenticationService authService, IConfigFileProvider configFileProvider) + public AuthenticationController(IAuthenticationService authService) { _authService = authService; - _configFileProvider = configFileProvider; } [HttpPost("login")] @@ -43,6 +41,7 @@ namespace Lidarr.Http.Authentication { IsPersistent = resource.RememberMe == "on" }; + await HttpContext.SignInAsync(AuthenticationType.Forms.ToString(), new ClaimsPrincipal(new ClaimsIdentity(claims, "Cookies", "user", "identifier")), authProperties); return Redirect("/"); diff --git a/src/Lidarr.Http/Frontend/Mappers/HtmlMapperBase.cs b/src/Lidarr.Http/Frontend/Mappers/HtmlMapperBase.cs index 85bd9acbd..1ad8a787f 100644 --- a/src/Lidarr.Http/Frontend/Mappers/HtmlMapperBase.cs +++ b/src/Lidarr.Http/Frontend/Mappers/HtmlMapperBase.cs @@ -39,15 +39,6 @@ namespace Lidarr.Http.Frontend.Mappers return stream; } - /* - public override IActionResult GetResponse(string resourceUrl) - { - var response = base.GetResponse(resourceUrl); - response.Headers["X-UA-Compatible"] = "IE=edge"; - - return response; - }*/ - protected string GetHtmlText() { if (RuntimeInfo.IsProduction && _generatedContent != null) diff --git a/src/Lidarr.Http/Middleware/CacheableSpecification.cs b/src/Lidarr.Http/Middleware/CacheableSpecification.cs index 21151b5a2..edcb846f1 100644 --- a/src/Lidarr.Http/Middleware/CacheableSpecification.cs +++ b/src/Lidarr.Http/Middleware/CacheableSpecification.cs @@ -39,12 +39,12 @@ namespace Lidarr.Http.Middleware return false; } - if (context.Request.Path.Equals("/index.js")) + if (context.Request.Path.Value?.EndsWith("/index.js") ?? false) { return false; } - if (context.Request.Path.Equals("/initialize.js")) + if (context.Request.Path.Value?.EndsWith("/initialize.js") ?? false) { return false; } @@ -55,7 +55,7 @@ namespace Lidarr.Http.Middleware } if (context.Request.Path.StartsWithSegments("/log", StringComparison.CurrentCultureIgnoreCase) && - context.Request.Path.ToString().EndsWith(".txt", StringComparison.CurrentCultureIgnoreCase)) + (context.Request.Path.Value?.EndsWith(".txt", StringComparison.CurrentCultureIgnoreCase) ?? false)) { return false; } diff --git a/src/Lidarr.Http/REST/Attributes/RestGetByIdAttribute.cs b/src/Lidarr.Http/REST/Attributes/RestGetByIdAttribute.cs index 47e75560b..7f9718ab8 100644 --- a/src/Lidarr.Http/REST/Attributes/RestGetByIdAttribute.cs +++ b/src/Lidarr.Http/REST/Attributes/RestGetByIdAttribute.cs @@ -1,21 +1,15 @@ using System; using System.Collections.Generic; -using Microsoft.AspNetCore.Mvc.Filters; using Microsoft.AspNetCore.Mvc.Routing; namespace Lidarr.Http.REST.Attributes { [AttributeUsage(AttributeTargets.Method)] - public class RestGetByIdAttribute : ActionFilterAttribute, IActionHttpMethodProvider, IRouteTemplateProvider + public class RestGetByIdAttribute : Attribute, IActionHttpMethodProvider, IRouteTemplateProvider { - public override void OnActionExecuting(ActionExecutingContext context) - { - Console.WriteLine($"OnExecuting {context.Controller.GetType()} {context.ActionDescriptor.DisplayName}"); - } - public IEnumerable HttpMethods => new[] { "GET" }; public string Template => "{id:int}"; - public new int? Order => 0; + public int? Order => 0; public string Name { get; } } }