Merge pull request #172 from marcominerva/develop

Make Roles optional in ApiKey and Credential classes

Author: marcominerva

src/SimpleAuthentication.Abstractions/ApiKey/ApiKey.cs

@@ -6,7 +6,7 @@ namespace SimpleAuthentication.ApiKey;
 /// <param name="Value">The API key value</param>
 /// <param name="UserName">The user name associated with the current key</param>
 /// <param name="Roles">The list of roles to assign to the user</param>
-public record class ApiKey(string Value, string UserName, IEnumerable<string> Roles)
+public record class ApiKey(string Value, string UserName, IEnumerable<string>? Roles = null)
 {
     /// <inheritdoc />
     public virtual bool Equals(ApiKey? other)
@@ -25,8 +25,5 @@ public virtual bool Equals(ApiKey? other)
     }
 
     /// <inheritdoc />
-    public override int GetHashCode()
-    {
-        return HashCode.Combine(Value, UserName);
-    }
+    public override int GetHashCode() => HashCode.Combine(Value, UserName);
 }

src/SimpleAuthentication.Abstractions/BasicAuthentication/Credential.cs

@@ -6,7 +6,7 @@ namespace SimpleAuthentication.BasicAuthentication;
 /// <param name="UserName">The user name</param>
 /// <param name="Password">The password</param>
 /// <param name="Roles">The list of roles to assign to the user</param>
-public record class Credential(string UserName, string Password, IEnumerable<string> Roles)
+public record class Credential(string UserName, string Password, IEnumerable<string>? Roles = null)
 {
     /// <inheritdoc />
     public virtual bool Equals(Credential? other)
@@ -25,8 +25,5 @@ public virtual bool Equals(Credential? other)
     }
 
     /// <inheritdoc />
-    public override int GetHashCode()
-    {
-        return HashCode.Combine(UserName, Password);
-    }
+    public override int GetHashCode() => HashCode.Combine(UserName, Password);
 }