Skip to content

Commit

Permalink
Merge pull request mozilla#1576 from mozilla/issue1575
Browse files Browse the repository at this point in the history
insert a setTimeout between two main phases of assertion generation

closes mozilla#1575
  • Loading branch information
benadida committed May 10, 2012
2 parents 31fdd88 + 71a5375 commit 77d24f3
Showing 1 changed file with 14 additions and 12 deletions.
26 changes: 14 additions & 12 deletions resources/static/shared/user.js
Original file line number Diff line number Diff line change
Expand Up @@ -943,18 +943,20 @@ BrowserID.User = (function() {
network.serverTime(function(serverTime) {
var sk = jwcrypto.loadSecretKeyFromObject(idInfo.priv);

// assertions are valid for 2 minutes
var expirationMS = serverTime.getTime() + (2 * 60 * 1000);
var expirationDate = new Date(expirationMS);

jwcrypto.assertion.sign(
{}, {audience: audience, expiresAt: expirationDate},
sk,
function(err, signedAssertion) {
assertion = jwcrypto.cert.bundle([idInfo.cert], signedAssertion);
storage.site.set(audience, "email", email);
complete(assertion);
});
setTimeout(function() {
// assertions are valid for 2 minutes
var expirationMS = serverTime.getTime() + (2 * 60 * 1000);
var expirationDate = new Date(expirationMS);

jwcrypto.assertion.sign(
{}, {audience: audience, expiresAt: expirationDate},
sk,
function(err, signedAssertion) {
assertion = jwcrypto.cert.bundle([idInfo.cert], signedAssertion);
storage.site.set(audience, "email", email);
complete(assertion);
});
}, 0);
}, onFailure);
}

Expand Down

0 comments on commit 77d24f3

Please sign in to comment.