themaster1970sf/TShock
1
0
Fork 0
Code Issues Pull requests Projects Releases 1 Packages Wiki Activity Actions

Turns out that OmniServer doesn't refactor after all, who knew? Fix build.

Browse source
This commit is contained in:
Lucas Nicodemus 2015-04-13 23:31:38 -06:00
parent b34c00107c
commit c5034a23ad
1 changed files with 7 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

12
TShockAPI/DB/UserManager.cs
View file

@ -355,16 +355,16 @@ namespace TShockAPI.DB
if (BCrypt.Net.BCrypt.Verify(password, this.Password))
{
// If necessary, perform an upgrade to the highest work factor.
upgradePasswordWorkFactor(password);
UpgradePasswordWorkFactor(password);
return true;
}
}
catch (SaltParseException)
{
if (hashPassword(password).ToUpper() == this.Password.ToUpper())
if (HashPassword(password).ToUpper() == this.Password.ToUpper())
{
// The password is not stored using BCrypt; upgrade it to BCrypt immediately
upgradePasswordToBCrypt(password);
UpgradePasswordToBCrypt(password);
return true;
}
return false;
@ -406,13 +406,15 @@ namespace TShockAPI.DB
protected internal void UpgradePasswordWorkFactor(string password)
{
// If the destination work factor is not greater, we won't upgrade it or re-hash it
int currentWorkFactor = 0;
try
{
int currentWorkFactor = Int32.Parse((this.Password.Split('$')[2]));
currentWorkFactor = Int32.Parse((this.Password.Split('$')[2]));
}
catch (FormatException)
{
TShock.Log.ConsoleError("Warning: Not upgrading work factor because bcrypt hash in an invalid format.");
return;
}
if (currentWorkFactor < TShock.Config.BCryptWorkFactor)
@ -510,7 +512,7 @@ namespace TShockAPI.DB
{
if (string.IsNullOrEmpty(password) || password == "non-existant password")
return null;
return hashPassword(Encoding.UTF8.GetBytes(password));
return HashPassword(Encoding.UTF8.GetBytes(password));
}
}