Report helpful error message for missing "jwt" dependency
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 48/100
- issue の種類
- 機能追加
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
- 技術スタック
- python
調査の方向性
まず、レポートに示されている JWTAuth.from_settings_file の呼び出しを追跡し、オプションの jwt 依存関係がどのように読み込まれるかを調べます。boxsdk[jwt] をインストールせずに依存関係がない場合を再現し、次にエラーで pip install boxsdk[jwt] を提案するようにして、依存関係がインストールされている場合も既存の JWT パスが引き続き機能することを確認します。
索引モデルが issue の本文から書いたものです。
説明
Is your feature request related to a problem? Please describe.
As is known from closed issue (https://github.com/box/box-python-sdk/issues/387) if you don't pip install "boxsdk[jwt]", and use the JWT authorization, you get a somewhat confusing message:
File "/Users/first.last/workspace/repo_name/box/../lib/box.py", line 70, in get_jwt_client
auth = boxsdk.JWTAuth.from_settings_file(load_config(self.app_name))
AttributeError: 'NoneType' object has no attribute 'from_settings_file'
Describe the solution you'd like
Return an error message instead that at least suggests "Did you try pip install boxsdk[jwt]?" when JWTAuth isn't going to work.
Additional context
Using sdk version boxsdk 2.9.0
- 主要言語
- Python
- スター
- 460
- フォーク
- 223
- 平均マージ
- 14時間 18分
- マージ済み PR(30日)
- 21
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
box/box-python-sdk のほかの issue
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
box/box-python-sdk#196 ·
-
bug enhancement
box/box-python-sdk#1314 · コメント 6 件 · 担当者 5 名 ·
-
enhancement
box/box-python-sdk#1154 · コメント 3 件 · 担当者 5 名 ·
-
enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
box/box-python-sdk#1155 · コメント 1 件 · リアクション 4 件 ·
-
enhancement
box/box-python-sdk#1157 · コメント 1 件 · 担当者 6 名 ·
box/box-python-sdk の issue をすべて見る
似ている issue
-
documentation help wanted
難易度 2/5 1〜3時間 初心者へのやさしさ 90/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 90/100
simonw/sqlite-utils#872 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100