Merge pull request #5 from dbarbera/master

removes moment dependency using built-in Date.now()
This commit is contained in:
Matias Woloski
2014-04-12 20:42:35 -03:00
2 changed files with 9 additions and 8 deletions
+8 -6
View File
@@ -1,5 +1,4 @@
var jws = require('jws');
var moment = require('moment');
module.exports.decode = function (jwt) {
return jws.decode(jwt).payload;
@@ -9,8 +8,13 @@ module.exports.sign = function(payload, secretOrPrivateKey, options) {
options = options || {};
var header = {typ: 'JWT', alg: options.algorithm || 'HS256'};
if (options.expiresInMinutes)
payload.exp = moment().add('minutes', options.expiresInMinutes).utc().unix();
payload.iat = Date.now();
if (options.expiresInMinutes) {
var ms = options.expiresInMinutes * 60 * 1000;
payload.exp = payload.iat + ms;
}
if (options.audience)
payload.aud = options.audience;
@@ -21,8 +25,6 @@ module.exports.sign = function(payload, secretOrPrivateKey, options) {
if (options.subject)
payload.sub = options.subject;
payload.iat = moment().utc().unix();
var signed = jws.sign({header: header, payload: payload, secret: secretOrPrivateKey});
return signed;
@@ -46,7 +48,7 @@ module.exports.verify = function(jwtString, secretOrPublicKey, options, callback
var payload = this.decode(jwtString);
if (payload.exp) {
if (moment().utc().unix() >= payload.exp)
if (Date.now() >= payload.exp)
return callback(new Error('jwt expired'));
}
+1 -2
View File
@@ -19,8 +19,7 @@
"url": "https://github.com/auth0/node-jsonwebtoken/issues"
},
"dependencies": {
"jws": "~0.2.2",
"moment": "~2.0.0"
"jws": "~0.2.2"
},
"devDependencies": {
"chai": "*",