Learn-json-web-tokens: Problema con el tiempo de vencimiento predeterminado
La intención es caducar en 7 días, pero el código no proporciona un número que esté 7 días antes de ahora ... ¡ni mucho menos!
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)
>
Usé esto en su lugar:
Date.now() + (1000 * 60 * 60 * 24 * 7)
alanshaw
Todos 6 comentarios
@alanshaw mucho mejor! : +1:
¿Tiene tiempo para presentarse como PR?
nelsonic
en 3 ago. 2015
PR # 78
emersonmellado
en 13 oct. 2017
Caballeros, mi lectura de this SO Pregunta / Respuesta:
https://stackoverflow.com/questions/28738168/how-to-create-jwt-exp-style-date-in-javascript
es que necesitamos _dividir_ por 1000 para convertir a _segundos_ por exp ...
nelsonic
en 14 oct. 2017
Creo que lo solucionamos desde el punto de vista de la caducidad agregando la opción '7d'.
Lo que veo ahora es que travis-ci no está contento con ese cambio y no estoy seguro de cómo depurarlo.
emersonmellado
en 20 oct. 2017
PR # 79 para corrección de prueba. construcción verde esta vez :)
https://travis-ci.org/dwyl/learn-json-web-tokens
emersonmellado
en 20 oct. 2017
@emersonmellado, gracias de nuevo por
nelsonic
en 20 oct. 2017
Temas relacionados
nelsonic
·
5Comentarios
KumarS-Naveen
·
3Comentarios
sarneeh
·
3Comentarios
joepie91
·
18Comentarios
NE-SmallTown
·
5Comentarios
Comentario más útil
PR # 79 para corrección de prueba. construcción verde esta vez :)
https://travis-ci.org/dwyl/learn-json-web-tokens