【发布时间】:2017-04-27 13:15:34
【问题描述】:
我现在已经实现了许多 TOTP 类,它们都生成了错误的输出。下面我发布了我用于最简单的代码。
我希望它能够像 Google Authenticator 一样实现和运行 - 例如代码 https://gauth.apps.gbraad.nl/#main。
所以我想要发生的是,在应用程序的前端,用户将输入他的秘密“BANANAKEY123”,它转换为“IJAU4QKOIFFUKWJRGIZQ====”的 base32 字符串。
现在在构造函数下面的键是“BANANAKEY123”。然而由于某种原因,它没有像 GAuth OTP 工具那样使用此代码生成相同的 OTP 密钥。
只有两个合理的错误是
var secretKeyBytes = Base32Encode(secretKey);
是错误的,或者我的计时功能是错误的。我检查了两者,都找不到任何错误。那么有人可以帮助我朝着正确的方向前进吗?谢谢!
public class Totp
{
private readonly int digits = 6;
private readonly HMACSHA1 hmac;
private readonly HMACSHA256 hmac256;
private readonly Int32 t1 = 30;
internal int mode;
private string secret;
private const string allowedCharacters = "ABCDEFGHIJKLMNOPQRSTUVWXYZ234567";
public Totp(string key, int mode)
{
secret = key;
this.mode = mode;
}
// defaults to SHA-1
public Totp(string key)
{
secret = key;
this.mode = 1;
}
public Totp(string base32string, Int32 t1, int digits) : this(base32string)
{
this.t1 = t1;
this.digits = digits;
}
public Totp(string base32string, Int32 t1, int digits, int mode) : this(base32string, mode)
{
this.t1 = t1;
this.digits = digits;
}
public String getCodeString()
{
return GetCode(this.secret, GetInterval(DateTime.UtcNow));
}
private static long GetInterval(DateTime dateTime)
{
TimeSpan elapsedTime = dateTime.ToUniversalTime() - new DateTime(1970, 1, 1, 0, 0, 0, DateTimeKind.Utc);
return (long)elapsedTime.TotalSeconds / 30;
}
private static string GetCode(string secretKey, long timeIndex)
{
var secretKeyBytes = Base32Encode(secretKey);
HMACSHA1 hmac = new HMACSHA1(secretKeyBytes);
byte[] challenge = BitConverter.GetBytes(timeIndex);
if (BitConverter.IsLittleEndian) Array.Reverse(challenge);
byte[] hash = hmac.ComputeHash(challenge);
int offset = hash[19] & 0xf;
int truncatedHash = hash[offset] & 0x7f;
for (int i = 1; i < 4; i++)
{
truncatedHash <<= 8;
truncatedHash |= hash[offset + i] & 0xff;
}
truncatedHash %= 1000000;
return truncatedHash.ToString("D6");
}
private static byte[] Base32Encode(string source)
{
var bits = source.ToUpper().ToCharArray().Select(c =>
Convert.ToString(allowedCharacters.IndexOf(c), 2).PadLeft(5, '0')).Aggregate((a, b) => a + b);
return Enumerable.Range(0, bits.Length / 8).Select(i => Convert.ToByte(bits.Substring(i * 8, 8), 2)).ToArray();
}
}
【问题讨论】:
-
顺便说一句,您的
Base32Encode方法效率非常低(ToUpper()和ToCharArray()调用是不必要的)并且使用Aggregate和String而不是StringBuilder会导致过度新的字符串分配(并在 O(n^n) 时间内运行)。 -
您的“密钥”中包含字符
1但这不是您的allowedCharacters设置中的一个错字吗?
标签: c# google-authentication one-time-password base32