Issue with default expiry time

[alanshaw]

The intention is to expire in 7 days, but the code doesn't provide a number thats 7 days ahead of now...far from it!
https://github.com/dwyl/learn-json-web-tokens/blob/master/example/lib/helpers.js#L39

$ node
> var x = Math.floor(new Date().getTime()/1000) + 7*24*60*60
undefined
> new Date(x)
Sat Jan 17 1970 15:46:41 GMT+0000 (BST)
> 

I used this instead:

Date.now() + (1000 * 60 * 60 * 24 * 7)

[nelsonic]

@alanshaw much betterer! 👍
Do you have time to submit as PR?

[emersonmellado]

PR out #78

[nelsonic]

Gents, my reading of this SO Question/Answer:
https://stackoverflow.com/questions/28738168/how-to-create-jwt-exp-style-date-in-javascript
is that we need to divide by 1000 to convert to seconds for exp ...

[emersonmellado]

I think we solved it from an expiration point of view by adding the '7d' option.

What I see now is that we have travis-ci not happy about that change and I'm not sure how to debug it.

[emersonmellado]

PR #79 out for test fix. green build this time :)

https://travis-ci.org/dwyl/learn-json-web-tokens

[nelsonic]

@emersonmellado thanks again for making time to fix this! 👍