code clean up

Author: damienbod

WebApi/SecurityHeadersDefinitions.cs

@@ -2,9 +2,15 @@
 
 public static class SecurityHeadersDefinitions
 {
+    private static HeaderPolicyCollection? policy;
+
     public static HeaderPolicyCollection GetHeaderPolicyCollection(bool isDev)
     {
-        var policy = new HeaderPolicyCollection()
+        // Avoid building a new HeaderPolicyCollection on every request for performance reasons.
+        // Where possible, cache and reuse HeaderPolicyCollection instances.
+        if (policy != null) return policy;
+
+        policy = new HeaderPolicyCollection()
             .AddFrameOptionsDeny()
             .AddContentTypeOptionsNoSniff()
             .AddReferrerPolicyStrictOriginWhenCrossOrigin()

WebHybridFlowClient/ApiService.cs

@@ -2,7 +2,7 @@
 using Microsoft.Extensions.Options;
 using System.Text.Json;
 
-namespace WebHybridClient;
+namespace WebHybridFlowClient;
 
 public class ApiService
 {

WebHybridFlowClient/ApiTokenCacheClient.cs

@@ -2,7 +2,7 @@
 using Microsoft.Extensions.Caching.Distributed;
 using Microsoft.Extensions.Options;
 
-namespace WebHybridClient;
+namespace WebHybridFlowClient;
 
 public class ApiTokenCacheClient
 {

WebHybridFlowClient/ApiTokenInMemoryClient.cs

@@ -2,7 +2,7 @@
 using Microsoft.Extensions.Options;
 using System.Collections.Concurrent;
 
-namespace WebHybridClient;
+namespace WebHybridFlowClient;
 
 public class ApiTokenInMemoryClient
 {

WebHybridFlowClient/AuthConfigurations.cs

@@ -1,4 +1,4 @@
-namespace WebHybridClient;
+namespace WebHybridFlowClient;
 
 public class AuthConfigurations
 {

WebHybridFlowClient/Controllers/HomeController.cs

@@ -1,9 +1,9 @@
 using Microsoft.AspNetCore.Authorization;
 using Microsoft.AspNetCore.Mvc;
 using System.Diagnostics;
-using WebHybridClient.Models;
+using WebHybridFlowClient.Models;
 
-namespace WebHybridClient.Controllers;
+namespace WebHybridFlowClient.Controllers;
 
 [Authorize]
 public class HomeController : Controller

WebHybridFlowClient/Controllers/TestController.cs

@@ -1,6 +1,6 @@
 using Microsoft.AspNetCore.Mvc;
 
-namespace WebHybridClient.Controllers;
+namespace WebHybridFlowClient.Controllers;
 
 [Route("[controller]")]
 public class StatusController : Controller

WebHybridFlowClient/HostingExtensions.cs

@@ -5,7 +5,7 @@
 using NetEscapades.AspNetCore.SecurityHeaders.Infrastructure;
 using Serilog;
 
-namespace WebHybridClient;
+namespace WebHybridFlowClient;
 
 internal static class HostingExtensions
 {

WebHybridFlowClient/Models/ErrorViewModel.cs

@@ -1,4 +1,4 @@
-namespace WebHybridClient.Models;
+namespace WebHybridFlowClient.Models;
 
 public class ErrorViewModel
 {

WebHybridFlowClient/Program.cs

@@ -1,6 +1,6 @@
 using Azure.Identity;
 using Serilog;
-using WebHybridClient;
+using WebHybridFlowClient;
 
 Log.Logger = new LoggerConfiguration()
     .WriteTo.Console()