Refactor AuthenticationOptions

Author: raman-m

src/Ocelot/Configuration/AuthenticationOptions.cs

@@ -1,43 +1,60 @@
 using Ocelot.Configuration.File;
-using System.Collections.Generic;
+using System;
 
 namespace Ocelot.Configuration
 {
     public sealed class AuthenticationOptions
     {
-        public AuthenticationOptions(List<string> allowedScopes, List<string> requiredRole, string authenticationProviderKey, string scopeKey, string roleKey, string policyName)
-        {
-            PolicyName = policyName;
-            AllowedScopes = allowedScopes;
-            RequiredRole = requiredRole;
-            AuthenticationProviderKey = authenticationProviderKey;
-            AuthenticationProviderKeys = Array.Empty<string>();
-        }
-
         public AuthenticationOptions(FileAuthenticationOptions from)
         {
             AllowedScopes = from.AllowedScopes ?? new();
-            AuthenticationProviderKey = from.AuthenticationProviderKey ?? string.Empty;
-            AuthenticationProviderKeys = from.AuthenticationProviderKeys ?? Array.Empty<string>();
+            BuildAuthenticationProviderKeys(from.AuthenticationProviderKey, from.AuthenticationProviderKeys);
+            PolicyName = from.PolicyName;
+            RequiredRole = from.RequiredRole;
+            ScopeKey = from.ScopeKey;
+            RoleKey = from.RoleKey;
         }
 
         public AuthenticationOptions(List<string> allowedScopes, string authenticationProviderKey, string[] authenticationProviderKeys)
         {
             AllowedScopes = allowedScopes ?? new();
-            AuthenticationProviderKey = authenticationProviderKey ?? string.Empty;
-            AuthenticationProviderKeys = authenticationProviderKeys ?? Array.Empty<string>();
+            BuildAuthenticationProviderKeys(authenticationProviderKey, authenticationProviderKeys);
         }
 
-        public AuthenticationOptions(List<string> allowedScopes, List<string> requiredRole, string authenticationProviderKey, string scopeKey, string roleKey, string policyName)
+        public AuthenticationOptions(List<string> allowedScopes, string[] authenticationProviderKeys, List<string> requiredRole, string scopeKey, string roleKey, string policyName)
         {
-            PolicyName = policyName;
             AllowedScopes = allowedScopes;
+            AuthenticationProviderKey = string.Empty;
+            AuthenticationProviderKeys = authenticationProviderKeys ?? Array.Empty<string>();
+            PolicyName = policyName;
             RequiredRole = requiredRole;
-            AuthenticationProviderKey = authenticationProviderKey;
             ScopeKey = scopeKey;
             RoleKey = roleKey;
         }
 
+        /// <summary>
+        /// Builds auth keys migrating legacy key to new ones.
+        /// </summary>
+        /// <param name="legacyKey">The legacy <see cref="AuthenticationProviderKey"/>.</param>
+        /// <param name="keys">New <see cref="AuthenticationProviderKeys"/> to build.</param>
+        private void BuildAuthenticationProviderKeys(string legacyKey, string[] keys)
+        {
+            keys ??= new string[];
+            if (string.IsNullOrEmpty(legacyKey))
+            {
+                return;
+            }
+
+            // Add legacy Key to new Keys array as the first element
+            var arr = new string[keys.Length + 1];
+            arr[0] = legacyKey;
+            Array.Copy(keys, 0, arr, 1, keys.Length);
+
+            // Update the object
+            AuthenticationProviderKeys = arr;
+            AuthenticationProviderKey = string.Empty;
+        }
+
         public List<string> AllowedScopes { get; }
 
         /// <summary>
@@ -47,7 +64,7 @@ public AuthenticationOptions(List<string> allowedScopes, List<string> requiredRo
         /// A <see langword="string"/> value of the scheme name.
         /// </value>
         [Obsolete("Use the " + nameof(AuthenticationProviderKeys) + " property!")]
-        public string AuthenticationProviderKey { get; }
+        public string AuthenticationProviderKey { get; private set; }
 
         /// <summary>
         /// Multiple authentication schemes registered in DI services with appropriate authentication providers.
@@ -58,7 +75,7 @@ public AuthenticationOptions(List<string> allowedScopes, List<string> requiredRo
         /// <value>
         /// An array of <see langword="string"/> values of the scheme names.
         /// </value>
-        public string[] AuthenticationProviderKeys { get; }
+        public string[] AuthenticationProviderKeys { get; private set; }
 
         public List<string> RequiredRole { get; }
         public string ScopeKey { get; }

src/Ocelot/Configuration/Builder/AuthenticationOptionsBuilder.cs

@@ -6,7 +6,6 @@ public class AuthenticationOptionsBuilder
     {
         private List<string> _allowedScopes = new();
         private List<string> _requiredRole = new();
-        private string _authenticationProviderKey;
         private string[] _authenticationProviderKeys = Array.Empty<string>();
         private string _roleKey;
         private string _scopeKey;
@@ -17,7 +16,7 @@ public AuthenticationOptionsBuilder WithAllowedScopes(List<string> allowedScopes
             _allowedScopes = allowedScopes;
             return this;
         }
-
+
         public AuthenticationOptionsBuilder WithRequiredRole(List<string> requiredRole)
         {
             _requiredRole = requiredRole;
@@ -26,10 +25,7 @@ public AuthenticationOptionsBuilder WithRequiredRole(List<string> requiredRole)
 
         [Obsolete("Use the " + nameof(WithAuthenticationProviderKeys) + " property!")]
         public AuthenticationOptionsBuilder WithAuthenticationProviderKey(string authenticationProviderKey)
-        {
-            _authenticationProviderKey = authenticationProviderKey;
-            return this;
-        }
+            => WithAuthenticationProviderKeys([authenticationProviderKey]);
 
         public AuthenticationOptionsBuilder WithAuthenticationProviderKeys(string[] authenticationProviderKeys)
         {
@@ -57,7 +53,7 @@ public AuthenticationOptionsBuilder WithPolicyName(string policyName)
 
         public AuthenticationOptions Build()
         {
-            return new AuthenticationOptions(_allowedScopes, _requiredRole, _authenticationProviderKey, _authenticationProviderKeys, _scopeKey, _roleKey, _policyName);
+            return new AuthenticationOptions(_allowedScopes, _authenticationProviderKeys, _requiredRole, _scopeKey, _roleKey, _policyName);
         }
     }
 }

src/Ocelot/Configuration/Creator/AuthenticationOptionsCreator.cs

@@ -6,7 +6,7 @@ public class AuthenticationOptionsCreator : IAuthenticationOptionsCreator
     {
         public AuthenticationOptions Create(FileRoute route)
         {
-            return new AuthenticationOptions(route.AuthenticationOptions.AllowedScopes, route.AuthenticationOptions.RequiredRole, route.AuthenticationOptions.AuthenticationProviderKey, route.AuthenticationOptions.ScopeKey, route.AuthenticationOptions.RoleKey, route.AuthenticationOptions.PolicyName);
+            return new AuthenticationOptions(route.AuthenticationOptions);
         }
     }
 }

src/Ocelot/Configuration/File/FileAuthenticationOptions.cs

@@ -22,7 +22,6 @@ public FileAuthenticationOptions(FileAuthenticationOptions from)
         public string AuthenticationProviderKey { get; set; }
         public string[] AuthenticationProviderKeys { get; set; }
 
-        public List<string> AllowedScopes { get; set; }
         public List<string> RequiredRole { get; set; }
         public string ScopeKey { get; set; }
         public string RoleKey { get; set; }
@@ -34,7 +33,7 @@ public FileAuthenticationOptions(FileAuthenticationOptions from)
             .Append($"{nameof(AllowedScopes)}:[{string.Join(',', AllowedScopes.Select(x => $"'{x}'"))}]")
             .ToString();
 
-        public override string ToString2()
+        public string ToString2()
         {
             var sb = new StringBuilder();
             sb.Append($"{nameof(AuthenticationProviderKey)}:{AuthenticationProviderKey},{nameof(AllowedScopes)}:[");