ghaiklor/passport-google-plus-token

Wrong or expired Google token cause error 500

Open

#50 aperta il 3 mar 2016

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)JavaScript (10 fork)github user discovery
bughelp wanted

Metriche repository

Star
 (23 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

If a token is expired or broken, system returns 500 instead of 401:

InternalOAuthError: Invalid Credentials at c:\node_modules\passport-google-plus-token\lib\index.js:115:25 at passBackControl (c:\node_modules\oauth\lib\oauth2.js:123:9) at IncomingMessage. (c:\node_modules\oauth\lib\oauth2.js:143:7) at emitNone (events.js:72:20) at IncomingMessage.emit (events.js:166:7) at endReadableNT (_stream_readable.js:905:12) at nextTickCallbackWith2Args (node.js:441:9) at process._tickCallback (node.js:355:17)

Guida contributor

Direzione di ricerca
Esamina la gestione degli errori della strategia passport google plus token in lib/index.js intorno alla riga 115. Modifica il callback per restituire uno stato HTTP 401 invece di 500 quando il token non è valido o è scaduto.
Tech stack
javascriptnodejs
Dominio
authenticationbackend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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
GitNode.jsOAuth
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65