graphql-python / graphql-python/graphene

401 status code for token authentication

オープン
#1,269 コメント 4 件 リアクション 8 件 担当者 0 名 GitHub で見る
主要言語
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

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

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