H-uru / dirtsand

The D'ni in Real-Time Server and Network Dæmon
GNU Affero General Public License v3.0
35 stars 24 forks source link

Two fixes for email-based authentication #181

Closed zrax closed 1 year ago

zrax commented 1 year ago

1) Fix the conditional for gametap domains to use the same regular expression that the client uses. It might have otherwise picked the wrong branch for some cases (e.g. "foo@bar" or "me@tld.gametap.com"). 2) Actually use BuggyHashPassword when creating an account with an email-based username. Previously, BuggyHashPassword wasn't ever used anywhere, and the SHA-1 hash was used unconditionally when adding accounts.