Make it clear that CacheControl is a function that mounts an adapter
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 5/5
- Tempo estimado
- Mais de uma semana
- Facilidade para iniciantes
- 35/100
- Tipo de issue
- Funcionalidade
- Clareza
- Razoavelmente clara
- Status de atividade
- Estagnada
- Stack de tecnologia
- python
- Domínio
- networking
Direção de pesquisa
Comece pela função CacheControl e inspecione como ela monta um HTTPAdapter na sessão para os prefixos http:// e https://. Decida se a mudança de nomenclatura da API solicitada ou um aviso de sobrescrita é pretendido e, em seguida, verifique se o comportamento escolhido torna explícita a substituição do adaptador.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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.
- Linguagem predominante
- Python
- Estrelas
- 505
- Forks
- 156
- Merge médio
- 5d 37min
- PRs com merge (30d)
- 2
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Sem modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de psf/cachecontrol
-
Cache Deserialization (part two) -- warning message incorrect?Talvez livre de novo Um pull request para esta issue foi fechado sem ser mesclado. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
psf/cachecontrol#209 · 5 comentários ·
-
Race condition in send()Talvez já em andamento @JulienPalard assumiu há 159 dias. Aberta
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 45/100
psf/cachecontrol#424 · 1 comentário ·
-
HTTP 302 Found responses with Cache Headers not being cachedTalvez já em andamento @cburroughs assumiu há 291 dias. Aberta
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 42/100
psf/cachecontrol#384 · 18 comentários ·
-
Head request is not cached even with cacheable_methods=("GET", HEAD") without a previous GETTalvez livre de novo Um pull request para esta issue foi fechado sem ser mesclado. Aberta
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 35/100
psf/cachecontrol#337 · 6 comentários ·
-
Investigate failure to load caches above `2^32 - 1` bytesTalvez livre de novo @woodruffw assumiu há 864 dias e não há nenhum pull request aberto. Aberta
psf/cachecontrol#336 · 4 comentários · 1 reação · 1 responsável ·
Todas as issues de psf/cachecontrol
Issues semelhantes
-
bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
conda-forge/conda-forge-bot#7025 ·
Mantenedores costumam responder em até 1 dia
-
area:ci
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 74/100
catincloud-labs/dagster-dex#149 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 84/100
home-assistant/core#188424 ·
Mantenedores costumam responder em até 1 dia
-
Review backlog (docs-currency): README 'CI run' link lands on a run whose overall badge is cancelledAbertaagent/reviewer needs-kind needs-triage
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 63/100
projectbluefin/utah#880 · 2 comentários ·
Mantenedores costumam responder em até 1 dia
-
LSan reports obj_gen leak on post-parse usage() exits (aarch64)Talvez já em andamento Um pull request vinculado a esta issue está aberto ou já foi mesclado. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 76/100
redis/memtier_benchmark#850 ·
Mantenedores costumam responder em até 1 dia