max-mapper / max-mapper/github-oauth
Consistent error handling
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- JavaScript
- Estrellas
- 71
- Forks
- 20
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
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.
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza en index.js#L45 y rastrea cómo la ruta de error de OAuth gestiona los callbacks y emite errores. Se considera terminado cuando un callback proporcionado recibe el error por la falta del código OAuth y el evento de error sigue emitiéndose con la respuesta.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- javascript, node.js
- Área
- authentication, backend
- Tipo de issue
- Error
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Estado de actividad
- Estancado
- Claridad
- Bien especificado
- Aptitud para principiantes
- 48/100