removes some old notes

src/Umbraco.Core/Security/AuthenticationExtensions.cs

@@ -307,11 +307,9 @@ namespace Umbraco.Core.Security
             }
             catch (Exception)
             {
-                //TODO: Do we need to do more here?? need to make sure that the forms cookie is gone, but is that
-                // taken care of in our custom middleware somehow?
                 ctx.Authentication.SignOut(
-                    Core.Constants.Security.BackOfficeAuthenticationType,
-                    Core.Constants.Security.BackOfficeExternalAuthenticationType);
+                    Constants.Security.BackOfficeAuthenticationType,
+                    Constants.Security.BackOfficeExternalAuthenticationType);
                 return null;
             }
         }

src/Umbraco.Core/Security/BackOfficeSignInManager.cs

@@ -46,7 +46,7 @@ namespace Umbraco.Core.Security
         /// </summary>
         /// <param name="userName"/><param name="password"/><param name="isPersistent"/><param name="shouldLockout"/>
         /// <returns/>
-        public async override Task<SignInStatus> PasswordSignInAsync(string userName, string password, bool isPersistent, bool shouldLockout)
+        public override async Task<SignInStatus> PasswordSignInAsync(string userName, string password, bool isPersistent, bool shouldLockout)
         {
             var result = await base.PasswordSignInAsync(userName, password, isPersistent, shouldLockout);
 

src/Umbraco.Core/Security/BackOfficeUserManager.cs

@@ -119,7 +119,7 @@ namespace Umbraco.Core.Security
             manager.ClaimsIdentityFactory = new BackOfficeClaimsIdentityFactory();
 
             manager.EmailService = new EmailService();
-
+            
             //NOTE: Not implementing these, if people need custom 2 factor auth, they'll need to implement their own UserStore to suport it
 
             //// Register two factor authentication providers. This application uses Phone and Emails as a step of receiving a code for verifying the user
@@ -200,7 +200,7 @@ namespace Umbraco.Core.Security
         /// We've allowed this check to be overridden with a simple callback so that developers don't actually
         /// have to implement/override this class.
         /// </remarks>
-        public async override Task<bool> CheckPasswordAsync(T user, string password)
+        public override async Task<bool> CheckPasswordAsync(T user, string password)
         {
             if (BackOfficeUserPasswordChecker != null)
             {