Learn-json-web-tokens: Problema com tempo de expiração padrão
A intenção é expirar em 7 dias, mas o código não fornece um número que falte 7 dias ... longe disso!
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)
>
Em vez disso, usei:
Date.now() + (1000 * 60 * 60 * 24 * 7)
alanshaw
Todos 6 comentários
@alanshaw muito melhor! : +1:
Você tem tempo para se inscrever como PR?
nelsonic
em 3 ago. 2015
PR out # 78
emersonmellado
em 13 out. 2017
Senhores, minha leitura de this SO Pergunta / Resposta:
https://stackoverflow.com/questions/28738168/how-to-create-jwt-exp-style-date-in-javascript
é que precisamos _dividir_ por 1000 para converter em _segundos_ para exp ...
nelsonic
em 14 out. 2017
Acho que resolvemos isso do ponto de vista da expiração, adicionando a opção '7d'.
O que vejo agora é que travis-ci não está feliz com essa mudança e não tenho certeza de como depurá-la.
emersonmellado
em 20 out. 2017
PR # 79 para correção de teste. construção verde desta vez :)
https://travis-ci.org/dwyl/learn-json-web-tokens
emersonmellado
em 20 out. 2017
@emersonmellado, obrigado novamente por arranjar tempo para consertar isso! 👍
nelsonic
em 20 out. 2017
Questões relacionadas
nelsonic
·
5Comentários
rhewitt22
·
5Comentários
sarneeh
·
3Comentários
rjmk
·
9Comentários
nelsonic
·
4Comentários
Comentários muito úteis
PR # 79 para correção de teste. construção verde desta vez :)
https://travis-ci.org/dwyl/learn-json-web-tokens