max-mapper / max-mapper/github-oauth

Consistent error handling

オープン
#10 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

主要言語
JavaScript
スター
71
フォーク
20
PR マージ指標
30日以内にマージされた PR はありません

説明

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.

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

index.js#L45 から開始し、OAuth のエラーパスがコールバックを処理してエラーを発行する仕組みを追跡します。提供されたコールバックが OAuth コードの欠落エラーを受け取り、エラーイベントがレスポンスとともに引き続き発行されれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
javascript, node.js
領域
authentication, backend
issue の種類
バグ
難易度
2/5
見積もり時間
1〜3時間
活発さ
停滞
明瞭さ
明確に書かれている
初心者へのやさしさ
48/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。