Attachment downloads ignore request URL and header overrides
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 39/100
Línea de trabajo
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.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- Python
- Estrellas
- 109
- Forks
- 85
- Merge medio
- 8 h 41 min
- PR fusionados (30 d)
- 3
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de nylas/nylas-python
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 48/100
nylas/nylas-python#455 ·
-
Bug: drafts.create() fails with TypeError for multipart requests (attachments > 3MB)Posiblemente ocupada @benjaminwhtan la tomó hace 266 días. Abierto
nylas/nylas-python#454 · 3 comentarios · 1 asignado ·
-
UnicodeEncodeError when sending e-mails with some iso-8859 characters (€ symbol)Quizá libre de nuevo @pengfeiye la tomó hace 277 días y no hay ningún pull request abierto. Abiertobug
nylas/nylas-python#448 · 1 comentario · 2 asignados ·
-
Create an async version of the clientQuizá libre de nuevo @AaronDDM la tomó hace 495 días y no hay ningún pull request abierto. Abiertoenhancement
nylas/nylas-python#412 · 1 comentario · 5 reacciones · 1 asignado ·
-
bug
Dificultad 3/5 1-2 días Aptitud para principiantes 38/100
nylas/nylas-python#402 · 1 reacción ·
Todos los issues de nylas/nylas-python
Issues similares
-
good first issue tools
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
Commits pushed after #1095 merged did not land (ahmi/unified-nova-mcp-review-e095 @ 80fa23a)Abiertobug deferred domain:ci P1
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
get_page_thumbnail modifies cached page images, affecting later thumbnails and picture exportsAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
docling-project/docling-mcp#147 ·
-
issue: A pipe model's Valves button ignores the Allow Chat Valves permissionPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abiertobug confirmed issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
open-webui/open-webui#32771 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Scheduled Security Scan is failing on mainPosiblemente ocupada @pkupt la tomó hoy. Abiertosecurity-scan-failure
Dificultad 1/5 1-3 horas Aptitud para principiantes 74/100
semantica-agi/semantica#2045 ·
Los mantenedores suelen responder en 2 días