dwyl/learn-json-web-tokens

Issue with default expiry time

Open

#43 aperta il 3 ago 2015

Vedi su GitHub
 (6 commenti) (0 reazioni) (0 assegnatari)JavaScript (267 fork)batch import
bughelp wanted

Metriche repository

Star
 (4175 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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)

Guida contributor

Direzione di ricerca
Esamina il file helpers.js e sostituisci il calcolo errato del timestamp con Date.now() + 7*24*60*60*1000.
Tech stack
javascript
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
90