Attachment downloads ignore request URL and header overrides
Avaliação
- Dificuldade
- 2/5
- Tempo estimado
- 1-3 horas
- Facilidade para iniciantes
- 39/100
Direção de pesquisa
Start by reading Client.attachments.download(...), download_bytes(...), and _execute_download_request, then check the _build_request argument order. Use the mocked requests.request boundary to verify both public methods honor the API URI and header overrides, omit the bodyless GET Content-type header, and report the overridden URL on timeout.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
Both Client.attachments.download(..., overrides=...) and download_bytes(...) pass their options through _execute_download_request, which currently calls _build_request("GET", path, headers, query_params, overrides) positionally. The fifth argument is request_body, not overrides.
For example, a call with overrides={"api_uri": "https://override.example.test", "headers": {"X-Test": "custom"}} still uses the original API URL and omits the custom header. Any nonempty overrides also cause a JSON Content-type header on the bodyless download GET. Timeout selection itself works, but a timeout error reports the original URL when the caller supplied a different API URI.
Reproduced using both public attachment download methods, the real HTTP client, and a mocked requests.request boundary. No live API or account calls. Fourteen regression/control cases:12 fail and2 pass on main. I have a focused keyword-argument fix and tests ready.
- Linguagem predominante
- Python
- Estrelas
- 109
- Forks
- 85
- Merge médio
- 8h 41min
- PRs com merge (30d)
- 3
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Tem um 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 nylas/nylas-python
-
bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 48/100
nylas/nylas-python#455 ·
-
Bug: drafts.create() fails with TypeError for multipart requests (attachments > 3MB)Talvez já em andamento @benjaminwhtan assumiu há 266 dias. Aberta
nylas/nylas-python#454 · 3 comentários · 1 responsável ·
-
UnicodeEncodeError when sending e-mails with some iso-8859 characters (€ symbol)Talvez livre de novo @pengfeiye assumiu há 277 dias e não há nenhum pull request aberto. Abertabug
nylas/nylas-python#448 · 1 comentário · 2 responsáveis ·
-
Create an async version of the clientTalvez livre de novo @AaronDDM assumiu há 495 dias e não há nenhum pull request aberto. Abertaenhancement
nylas/nylas-python#412 · 1 comentário · 5 reações · 1 responsável ·
-
bug
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 38/100
nylas/nylas-python#402 · 1 reação ·
Todas as issues de nylas/nylas-python
Issues semelhantes
-
get_page_thumbnail modifies cached page images, affecting later thumbnails and picture exportsAberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
docling-project/docling-mcp#147 ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 80/100
Mantenedores costumam responder em até 1 dia
-
enhancement
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 76/100
autowarefoundation/autoware_carla_scenario#405 ·
Mantenedores costumam responder em até 1 dia
-
bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 79/100
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
mohitmishra786/readprism#322 ·
Mantenedores costumam responder em até 1 dia