Add override option for IsAuthenticated for GarnetAclWithAadAuthentic…

…ator to respect token validation. (#868)

* Override for AclWithAadAuthenticator added. Earlier it would only use IsAuthenticated from base class i.e. GarnetAclAuthenticator which would not abide by IsAuthenticated from aad authenticator which sets token validity.

* Fixing whitespace

libs/server/Auth/GarnetACLAuthenticator.cs

@@ -44,7 +44,7 @@ public GarnetACLAuthenticator(AccessControlList accessControlList, ILogger logge
         /// <summary>
         /// Check if the user is authorized to execute commands.
         /// </summary>
-        public bool IsAuthenticated => _user != null;
+        public virtual bool IsAuthenticated => _user != null;
 
         /// <summary>
         /// ACL authenticator is can use ACL.

libs/server/Auth/GarnetAclWithAadAuthenticator.cs

@@ -38,5 +38,13 @@ protected override bool AuthenticateInternal(User user, ReadOnlySpan<byte> usern
             }
             return false;
         }
+
+        public override bool IsAuthenticated
+        {
+            get
+            {
+                return this._garnetAuthenticator.IsAuthenticated && base.IsAuthenticated;
+            }
+        }
     }
 }