Make it clear that CacheControl is a function that mounts an adapter
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Accessibilité débutants
- 35/100
- Type d'issue
- Fonctionnalité
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- python
- Domaine
- networking
Piste de recherche
Commencez par la fonction CacheControl et examinez comment elle monte un HTTPAdapter sur la session pour les préfixes http:// et https://. Déterminez si le changement de nommage de l’API demandé ou un avertissement d’écrasement est prévu, puis vérifiez que le comportement choisi rend explicite le remplacement de l’adaptateur.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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.
- Langage dominant
- Python
- Étoiles
- 505
- Forks
- 153
- Merge moyen
- 5 j 6 h
- PR mergées (30 j)
- 5
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de psf/cachecontrol
-
Cache Deserialization (part two) -- warning message incorrect?Peut-être à nouveau libre Une pull request pour cette issue a été fermée sans être fusionnée. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
psf/cachecontrol#209 · 5 commentaires ·
-
Race condition in send()Peut-être pris @JulienPalard l’a pris il y a 149 jours. Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 45/100
psf/cachecontrol#424 · 1 commentaire ·
-
HTTP 302 Found responses with Cache Headers not being cachedPeut-être pris @cburroughs l’a pris il y a 281 jours. Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 42/100
psf/cachecontrol#384 · 18 commentaires ·
-
Head request is not cached even with cacheable_methods=("GET", HEAD") without a previous GETPeut-être à nouveau libre Une pull request pour cette issue a été fermée sans être fusionnée. Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
psf/cachecontrol#337 · 6 commentaires ·
-
Investigate failure to load caches above `2^32 - 1` bytesPeut-être à nouveau libre @woodruffw l’a pris il y a 854 jours, et aucune pull request n’est ouverte. Ouverte
psf/cachecontrol#336 · 4 commentaires · 1 réaction · 1 personne assignée ·
Toutes les issues de psf/cachecontrol
Issues similaires
-
`LazyFrameRemote` and `ExecuteRemote` sinks do not accept `Path`, like `LazyFrame` sinks do.Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
WingedGuardian/GENesis-AGI#3375 ·
Les mainteneurs répondent en général sous 1 jour
-
curriculum documentation quality
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
githubnext/gh-aw-workshop#4601 ·
Les mainteneurs répondent en général sous 1 jour
-
approved Good First Issue scope: docs
Difficulté 1/5 Moins d'une heure Accessibilité débutants 85/100
hiero-ledger/hiero-sdk-python#2730 · 3 commentaires ·
Les mainteneurs répondent en général sous 1 jour