max-mapper / max-mapper/github-oauth
Consistent error handling
Personne n'a encore pris cette issue.
- Langage dominant
- JavaScript
- Étoiles
- 71
- Forks
- 20
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
Hi! What do you think about changing https://github.com/maxogden/github-oauth/blob/master/index.js#L45 from
if (!code) return emitter.emit('error', {error: 'missing oauth code'}, resp)
to something like this:
if (!code) {
var err = {error: 'missing oauth code'}
if (cb) {
cb(err)
}
return emitter.emit('error', err, resp)
}
because now if I rely on a callback that is the case when my callback won't be triggered.
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez à index.js#L45 et suivez la manière dont le chemin de gestion des erreurs OAuth traite les callbacks et émet les erreurs. C’est terminé lorsqu’un callback fourni reçoit l’erreur indiquant que le code OAuth est manquant et que l’événement d’erreur continue d’être émis avec la réponse.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- javascript, node.js
- Domaine
- authentication, backend
- Type d'issue
- Bug
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Activité
- À l'abandon
- Clarté
- Clairement spécifiée
- Accessibilité débutants
- 48/100