exp should be unix timestamp in seconds, not milliseconds

This commit is contained in:
Matias Woloski
2014-05-03 08:29:06 -03:00
parent 145c01440d
commit 44ed992ee5
2 changed files with 4 additions and 4 deletions
+3 -3
View File
@@ -9,10 +9,10 @@ module.exports.sign = function(payload, secretOrPrivateKey, options) {
var header = {typ: 'JWT', alg: options.algorithm || 'HS256'};
payload.iat = Date.now();
payload.iat = Math.round(Date.now() / 1000);
if (options.expiresInMinutes) {
var ms = options.expiresInMinutes * 60 * 1000;
var ms = options.expiresInMinutes * 60;
payload.exp = payload.iat + ms;
}
@@ -48,7 +48,7 @@ module.exports.verify = function(jwtString, secretOrPublicKey, options, callback
var payload = this.decode(jwtString);
if (payload.exp) {
if (Date.now() >= payload.exp)
if (Math.round(Date.now()) / 1000 >= payload.exp)
return callback(new Error('jwt expired'));
}
+1 -1
View File
@@ -1,6 +1,6 @@
{
"name": "jsonwebtoken",
"version": "0.3.0",
"version": "0.4.0",
"description": "JSON Web Token implementation (symmetric and asymmetric)",
"main": "index.js",
"scripts": {