Make it clear that CacheControl is a function that mounts an adapter
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Anfängerfreundlichkeit
- 35/100
- Issue-Typ
- Feature
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Veraltet
- Tech-Stack
- python
- Bereich
- networking
Rechercherichtung
Beginne bei der Funktion CacheControl und prüfe, wie sie einen HTTPAdapter für die Präfixe http:// und https:// in der Session einbindet. Entscheide, ob die angeforderte Änderung der API-Benennung oder eine Warnung beim Überschreiben vorgesehen ist, und verifiziere anschließend, dass das gewählte Verhalten den Austausch des Adapters ausdrücklich macht.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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.
- Vorherrschende Sprache
- Python
- Sterne
- 505
- Forks
- 153
- Ø Merge
- 5 T. 6 Std.
- Gemergte PRs (30 T.)
- 5
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus psf/cachecontrol
-
Cache Deserialization (part two) -- warning message incorrect?Evtl. wieder frei Ein Pull Request dazu wurde ohne Merge geschlossen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
psf/cachecontrol#209 · 5 Kommentare ·
-
Race condition in send()Evtl. vergeben @JulienPalard hat das vor 150 Tagen übernommen. Offen
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 45/100
psf/cachecontrol#424 · 1 Kommentar ·
-
HTTP 302 Found responses with Cache Headers not being cachedEvtl. vergeben @cburroughs hat das vor 282 Tagen übernommen. Offen
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 42/100
psf/cachecontrol#384 · 18 Kommentare ·
-
Head request is not cached even with cacheable_methods=("GET", HEAD") without a previous GETEvtl. wieder frei Ein Pull Request dazu wurde ohne Merge geschlossen. Offen
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 35/100
psf/cachecontrol#337 · 6 Kommentare ·
-
Investigate failure to load caches above `2^32 - 1` bytesEvtl. wieder frei @woodruffw hat das vor 855 Tagen übernommen, und es ist kein Pull Request offen. Offen
psf/cachecontrol#336 · 4 Kommentare · 1 Reaktion · 1 zugewiesene Person ·
Alle Issues in psf/cachecontrol
Ähnliche Issues
-
area:core-platform effort:S priority:P3 status:in-review type:test
Schwierigkeit 2/5 Unter einer Stunde Anfängerfreundlichkeit 80/100
randomparity/kdive#3246 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Requantification library converts m/z to mass with the hydrogen-atom mass instead of the proton massOffenbug
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 85/100
nf-core/metaboigniter#130 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
common-workflow-language/cwl-utils#477 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Add dependabotOffenfeat
Schwierigkeit 2/5 Unter einer Stunde Anfängerfreundlichkeit 72/100
open-nudge/opentemplate#204 ·
Maintainer antworten meist innerhalb von 1 Tag
-
github-stale-ci-pr-closer: KV state is double-wrapped, never persists, and hits the depth-32 limitEvtl. vergeben @VascoSch92 hat das heute übernommen. Offenpriority:medium ready-for-dev
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
OpenHands/extensions#781 ·
Maintainer antworten meist innerhalb von 1 Tag