Skip to content

Commit d7c1dac

Browse files
dyzymartincostello
authored andcommitted
Odnoklassniki provider: Code cleanup for skipped conversations
1 parent 0d9bc1f commit d7c1dac

File tree

1 file changed

+9
-7
lines changed

1 file changed

+9
-7
lines changed

src/AspNet.Security.OAuth.Odnoklassniki/OdnoklassnikiAuthenticationHandler.cs

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -40,8 +40,8 @@ protected override async Task<AuthenticationTicket> CreateTicketAsync([NotNull]
4040
string sign;
4141
using (var algorithm = MD5.Create())
4242
{
43-
var accessSecret = GetMd5Hash(algorithm, tokens.AccessToken + Options.ClientSecret);
44-
sign = GetMd5Hash(algorithm, $"application_key={Options.PublicSecret}format=jsonmethod=users.getCurrentUser{accessSecret}");
43+
var accessSecret = GetHash(algorithm, tokens.AccessToken + Options.ClientSecret);
44+
sign = GetHash(algorithm, $"application_key={Options.PublicSecret}format=jsonmethod=users.getCurrentUser{accessSecret}");
4545
}
4646

4747
var address = QueryHelpers.AddQueryString(Options.UserInformationEndpoint, new Dictionary<string, string>
@@ -78,12 +78,14 @@ protected override async Task<AuthenticationTicket> CreateTicketAsync([NotNull]
7878
return new AuthenticationTicket(context.Principal, context.Properties, Scheme.Name);
7979
}
8080

81-
private static string GetMd5Hash(HashAlgorithm hashAlgorithm, string input)
81+
private static string GetHash(HashAlgorithm algorithm, string input)
8282
{
83-
var sBuilder = new StringBuilder();
84-
foreach (var t in hashAlgorithm.ComputeHash(Encoding.UTF8.GetBytes(input)))
85-
sBuilder.Append(t.ToString("x2", CultureInfo.InvariantCulture));
86-
return sBuilder.ToString();
83+
var builder = new StringBuilder();
84+
foreach (var b in algorithm.ComputeHash(Encoding.UTF8.GetBytes(input)))
85+
{
86+
builder.Append(b.ToString("x2", CultureInfo.InvariantCulture));
87+
}
88+
return builder.ToString();
8789
}
8890
}
8991
}

0 commit comments

Comments
 (0)