diff --git a/Projects/UOContent/Accounting/AccountHandler.cs b/Projects/UOContent/Accounting/AccountHandler.cs index 17ee2f809..4f32b3e62 100644 --- a/Projects/UOContent/Accounting/AccountHandler.cs +++ b/Projects/UOContent/Accounting/AccountHandler.cs @@ -398,8 +398,12 @@ public static class AccountHandler /// /// Hands the password check to the worker, whatever algorithm it uses. Every protection is safe - /// to run off the loop, so there is no carve-out; a cheap digest pays a thread hop it does not - /// need, but login latency is not what this is protecting. + /// to run off the loop, so there is no carve-out. + /// + /// Nor is a cheap digest worth carving out. AccountSecurity.Configure refuses anything + /// below SHA2 as the configured algorithm, so MD5 and SHA1 only ever appear as a stored hash + /// awaiting migration -- which makes NeedsPasswordUpgrade true, and the job carries the + /// upgrade hash that dominates it. The microsecond digest is never the whole job. /// private static PasswordCheckDispatch DispatchPasswordCheck(AccountLoginEventArgs e, Account acct, string pw) {