max-mapper / max-mapper/github-oauth
Consistent error handling
オープン
まだ誰も着手していません。
- 主要言語
- 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.
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
index.js#L45 から開始し、OAuth のエラーパスがコールバックを処理してエラーを発行する仕組みを追跡します。提供されたコールバックが OAuth コードの欠落エラーを受け取り、エラーイベントがレスポンスとともに引き続き発行されれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- javascript, node.js
- 領域
- authentication, backend
- issue の種類
- バグ
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 活発さ
- 停滞
- 明瞭さ
- 明確に書かれている
- 初心者へのやさしさ
- 48/100