codefactor fixes

Author: petrsvihlik

sample/WopiHost.Validator/Infrastructure/NonSecureSecurityToken.cs

@@ -0,0 +1,44 @@
+using Microsoft.IdentityModel.Tokens;
+
+namespace WopiHost.Validator.Infrastructure;
+
+public class NonSecureSecurityToken : SecurityToken
+{
+    private readonly string _id;
+    private readonly string _userName;
+    private readonly DateTime _effectiveTime;
+
+    public NonSecureSecurityToken(string userName)
+    {
+        _userName = userName;
+        _id = userName;// Guid.NewGuid().ToString("N");
+        _effectiveTime = DateTime.UtcNow;
+    }
+
+    public override string Id
+    {
+        get { return _id; }
+    }
+
+    public override DateTime ValidFrom
+    {
+        get { return _effectiveTime; }
+    }
+
+    public override DateTime ValidTo
+    {
+        // Never expire
+        get { return DateTime.MaxValue; }
+    }
+
+    public string UserName
+    {
+        get { return _userName; }
+    }
+
+    public override string Issuer => throw new NotImplementedException();
+
+    public override SecurityKey SecurityKey => throw new NotImplementedException();
+
+    public override SecurityKey SigningKey { get => throw new NotImplementedException(); set => throw new NotImplementedException(); }
+}

sample/WopiHost.Validator/Infrastructure/WopiSecurityHandler.cs

@@ -60,44 +60,3 @@ public Task<bool> IsAuthorized(ClaimsPrincipal principal, IWopiAuthorizationRequ
     /// </summary>
     public string WriteToken(SecurityToken token) => token.Id;
 }
-
-public class NonSecureSecurityToken : SecurityToken
-{
-    private readonly string _id;
-    private readonly string _userName;
-    private readonly DateTime _effectiveTime;
-
-    public NonSecureSecurityToken(string userName)
-    {
-        _userName = userName;
-        _id = userName;// Guid.NewGuid().ToString("N");
-        _effectiveTime = DateTime.UtcNow;
-    }
-
-    public override string Id
-    {
-        get { return _id; }
-    }
-
-    public override DateTime ValidFrom
-    {
-        get { return _effectiveTime; }
-    }
-
-    public override DateTime ValidTo
-    {
-        // Never expire
-        get { return DateTime.MaxValue; }
-    }
-
-    public string UserName
-    {
-        get { return _userName; }
-    }
-
-    public override string Issuer => throw new NotImplementedException();
-
-    public override SecurityKey SecurityKey => throw new NotImplementedException();
-
-    public override SecurityKey SigningKey { get => throw new NotImplementedException(); set => throw new NotImplementedException(); }
-}

src/WopiHost.Core/Controllers/EcosystemController.cs

@@ -22,23 +22,23 @@ namespace WopiHost.Core.Controllers;
 public class EcosystemController(
     IWopiStorageProvider storageProvider) : ControllerBase
 {
-	/// <summary>
-	/// The GetRootContainer operation returns the root container. A WOPI client can use this operation to get a reference to the root container, from which the client can call EnumerateChildren (containers) to navigate a container hierarchy.
-	/// Specification: https://learn.microsoft.com/microsoft-365/cloud-storage-partner-program/rest/ecosystem/getrootcontainer
-	/// Example URL: GET /wopi/ecosystem/root_container_pointer
-	/// </summary>
-	/// <returns></returns>
-	[HttpGet("root_container_pointer")]
-	[Produces(MediaTypeNames.Application.Json)]
-	public RootContainerInfo GetRootContainer() //TODO: fix the path
-	{
-		var root = storageProvider.GetWopiContainer(@".\");
-		var rc = new RootContainerInfo
-		{
-			ContainerPointer = new ChildContainer(root.Name, Url.GetWopiUrl(WopiResourceType.Container, root.Identifier))
-		};
-		return rc;
-	}
+    /// <summary>
+    /// The GetRootContainer operation returns the root container. A WOPI client can use this operation to get a reference to the root container, from which the client can call EnumerateChildren (containers) to navigate a container hierarchy.
+    /// Specification: https://learn.microsoft.com/microsoft-365/cloud-storage-partner-program/rest/ecosystem/getrootcontainer
+    /// Example URL: GET /wopi/ecosystem/root_container_pointer
+    /// </summary>
+    /// <returns></returns>
+    [HttpGet("root_container_pointer")]
+    [Produces(MediaTypeNames.Application.Json)]
+    public RootContainerInfo GetRootContainer() //TODO: fix the path
+    {
+        var root = storageProvider.GetWopiContainer(@".\");
+        var rc = new RootContainerInfo
+        {
+            ContainerPointer = new ChildContainer(root.Name, Url.GetWopiUrl(WopiResourceType.Container, root.Identifier))
+        };
+        return rc;
+    }
 
     /// <summary>
     /// The CheckEcosystem operation is similar to the the CheckFileInfo operation, but does not require a file or container ID.

src/WopiHost.Url/WopiUrlBuilder.cs

@@ -61,5 +61,4 @@ public partial class WopiUrlBuilder(IDiscoverer discoverer, WopiUrlSettings? url
         }
         return null;
     }
-
 }