Make it clear that CacheControl is a function that mounts an adapter
まだ誰も着手していません。
評価
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 初心者へのやさしさ
- 35/100
- issue の種類
- 機能追加
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
- 技術スタック
- python
- 領域
- networking
調査の方向性
CacheControl 関数から始め、http:// および https:// プレフィックスに対してセッションへ HTTPAdapter をマウントする方法を調べます。要求された API の命名変更と上書き警告のどちらが意図されているかを判断し、選択した動作によってアダプターの置き換えが明示的になることを確認します。
索引モデルが issue の本文から書いたものです。
説明
The camel-case notation suggests that CacheControl is a class. Given the docs terminology, according to which CacheControl is a "wrapper", I expect that CacheControl returns an object that wraps a session.
But CacheControl is neither a class nor it returns a wrapper object. It's a function which mounts an HTTPAdapter to the session and returns the session itself. It's a function with side-effects that could even return None.
My problem with it is that it's not even innocently misleading, because if the user has already mounted an HTTPAdapter for "http://" and "https://", CacheControl will overwrite that adapter without the user even realizing what happened because he/she was convinced by the naming convention and the docs that CacheControl is a wrapper.
Consider:
- switching to a name that reflects what's actually happening, e.g.
mount_cache_adapter; - warning the user that any eventual adapter for those prefixes will be overwritten.
- 主要言語
- Python
- スター
- 505
- フォーク
- 153
- 平均マージ
- 5日 6時間
- マージ済み PR（30日）
- 5
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
psf/cachecontrol のほかの issue
-
Cache Deserialization (part two) -- warning message incorrect?再び着手できるかも このイシューのプルリクエストはマージされずにクローズされました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
psf/cachecontrol#209 · コメント 5 件 ·
-
Race condition in send()対応中かも @JulienPalard が 149 日前に担当しました。 オープン
難易度 4/5 3〜5日 初心者へのやさしさ 45/100
psf/cachecontrol#424 · コメント 1 件 ·
-
HTTP 302 Found responses with Cache Headers not being cached対応中かも @cburroughs が 281 日前に担当しました。 オープン
難易度 3/5 1〜2日 初心者へのやさしさ 42/100
psf/cachecontrol#384 · コメント 18 件 ·
-
Head request is not cached even with cacheable_methods=("GET", HEAD") without a previous GET再び着手できるかも このイシューのプルリクエストはマージされずにクローズされました。 オープン
難易度 3/5 1〜2日 初心者へのやさしさ 35/100
psf/cachecontrol#337 · コメント 6 件 ·
-
Investigate failure to load caches above `2^32 - 1` bytes再び着手できるかも @woodruffw が 854 日前に担当しましたが、オープン中のプルリクエストはありません。 オープン
psf/cachecontrol#336 · コメント 4 件 · リアクション 1 件 · 担当者 1 名 ·
psf/cachecontrol の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
WingedGuardian/GENesis-AGI#3375 ·
メンテナーはふだん 1 日以内に返信
-
curriculum documentation quality
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
githubnext/gh-aw-workshop#4601 ·
メンテナーはふだん 1 日以内に返信
-
approved Good First Issue scope: docs
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
hiero-ledger/hiero-sdk-python#2730 · コメント 3 件 ·
メンテナーはふだん 1 日以内に返信