Skip to content
This repository has been archived by the owner on Mar 20, 2019. It is now read-only.

Commit

Permalink
Fixed bug in establishing associations with cleartext secrets.
Browse files Browse the repository at this point in the history
  • Loading branch information
AArnott committed Mar 18, 2008
1 parent 2992d2c commit 485d5cd
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/DotNetOpenId/Consumer/AssociateResponse.cs
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,9 @@ public Association Association {
byte[] secret;

string session_type;
if (!Args.TryGetValue(QueryStringArgs.openidnp.session_type, out session_type)) {
if (!Args.TryGetValue(QueryStringArgs.openidnp.session_type, out session_type) ||
session_type == QueryStringArgs.plaintext ||
session_type == string.Empty) {
secret = getDecoded(QueryStringArgs.mac_key);
} else if (QueryStringArgs.DH_SHA1.Equals(session_type, StringComparison.Ordinal)) {
byte[] dh_server_public = getDecoded(QueryStringArgs.openidnp.dh_server_public);
Expand Down

0 comments on commit 485d5cd

Please sign in to comment.