graphql-python / graphql-python/graphene
401 status code for token authentication
- 主要言語
- Python
- スター
- 8.2k
- フォーク
- 818
- PR マージ指標
- 30日以内にマージされた PR はありません
説明
I think in the case of token authentication, it would make sense to be able to return a 401, **even if** only one field errors out this way (referring to the example given in the original post). This status code is necessary to make this library compatible with authMiddleware in [React Relay Network Modern](https://github.com/relay-tools/react-relay-network-modern), which expects a 401 status code to refresh tokens.
_Originally posted by @berkcoker in https://github.com/graphql-python/graphene/issues/946#issuecomment-698617718_
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
調査の方向性
まず、リンク先の元の issue と React Relay Network Modern の authMiddleware 要件を読みます。Graphene におけるトークン認証のエラーパスを追跡し、1 つのフィールドでエラーが発生した場合に 401 をどのように適用できるかを判断します。回帰テストによるカバレッジを含めて動作が実装され、通常の GraphQL エラーレスポンスとの互換性が維持されれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- graphql, python
- 領域
- api, authentication
- issue の種類
- 機能追加
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 活発さ
- 停滞
- 明瞭さ
- 説明が足りない
- 初心者へのやさしさ
- 30/100