Sr at21 functional test (#265)

* small stuff * quicktest
Altinn · May 22, 2024 · d01dc95 · d01dc95
1 parent f2462ad
commit d01dc95
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 7 deletions.
diff --git a/...n.Authentication.UI/Altinn.Authentication.UI.Core/SystemRegister/SystemRegisterService.cs b/...n.Authentication.UI/Altinn.Authentication.UI.Core/SystemRegister/SystemRegisterService.cs
@@ -11,9 +11,9 @@ public SystemRegisterService(ISystemRegisterClient systemRegisterClient)
 
     public async Task<List<RegisteredSystemDTO>> GetListRegSys(CancellationToken cancellationToken)
     {
-        List<RegisteredSystemDTO> lista = new();
+        List<RegisteredSystemDTO> lista = [];
 
-        lista = await _systemRegisterClient.GetListRegSys();
+        lista = await _systemRegisterClient.GetListRegSys(cancellationToken );
 
         return lista;
     }

diff --git a/bff/src/Altinn.Authentication.UI/Altinn.Authentication.UI/Controllers/HomeController.cs b/bff/src/Altinn.Authentication.UI/Altinn.Authentication.UI/Controllers/HomeController.cs
@@ -11,12 +11,15 @@
 
 namespace Altinn.Authentication.UI.Controllers;
 
+/// <summary>
+/// The Home controller
+/// </summary>
 [Route("authfront/")]
 [Route("authfront/ui")]
 [Route("authfront/ui/{*AnyValue}")]
 public class HomeController : Controller
 {
-    //private readonly ILogger<HomeController> _logger;
+    private readonly ILogger<HomeController> _logger;
     private readonly IAntiforgery _antiforgery;
     private readonly IWebHostEnvironment _env;
     private readonly IHttpContextAccessor _httpContextAccessor;
@@ -34,16 +37,17 @@ public class HomeController : Controller
     /// <param name="profileService">service implementation for user profile</param>
     /// <param name="httpContextAccessor">http context</param>
     /// <param name="generalSettings">general settings</param>
+    /// <param name="logger">logger</param>
     public HomeController(
         IUserProfileService profileService,
         IAntiforgery antiforgery,
         IWebHostEnvironment env,
         IHttpContextAccessor httpContextAccessor,
         IOptions<PlatformSettings> platformSettings,
-        IOptions<GeneralSettings> generalSettings)
-        //ILogger<HomeController> logger)
+        IOptions<GeneralSettings> generalSettings,
+        ILogger<HomeController> logger)
     {
-        //_logger = logger;
+        _logger = logger;
         _antiforgery = antiforgery;
         _env = env;
         _httpContextAccessor = httpContextAccessor;

diff --git a/...tinn.Authentication.UI/Filters/ValidateAntiforgeryTokenIfAuthCookieAuthorizationFilter.cs b/...tinn.Authentication.UI/Filters/ValidateAntiforgeryTokenIfAuthCookieAuthorizationFilter.cs
@@ -51,7 +51,7 @@ protected virtual bool ShouldValidate(AuthorizationFilterContext context)
 
         string method = context.HttpContext.Request.Method;
         if (
-            //string.Equals("GET", method, StringComparison.OrdinalIgnoreCase) ||  //Why ?
+            string.Equals("GET", method, StringComparison.OrdinalIgnoreCase) ||  //Why ?
             string.Equals("HEAD", method, StringComparison.OrdinalIgnoreCase) ||
             string.Equals("TRACE", method, StringComparison.OrdinalIgnoreCase) ||
             string.Equals("OPTIONS", method, StringComparison.OrdinalIgnoreCase)