Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

Tool, service and variable get requests omit configured headers

Abierto Apto para principiantes
  • ¿Algo no está bien?
#122 0 comentarios 0 reacciones 0 asignados Ver en GitHub

@sylvesterkaczmarek ya está trabajando en esto.

Desde el 26/9/2026.

  • #124 de @sylvesterkaczmarek — abierto

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
88/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
python
Área
api, backend

Línea de trabajo

Comienza en ToolServiceVariableBaseQuery.get y compara su bucle de solicitudes con Query.get, Query.results y Query.hits. Usa la interceptación de send de Requests para verificar los encabezados configurados en ToolQuery, ServiceQuery y VariableQuery, incluida la paginación y los resultados vacíos. Se considera terminado cuando pasan los casos offline específicos, mientras que la paginación, el formato de salida y la propagación de errores HTTP permanecen sin cambios.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Problem

On develop at 9f7648ccb925b9c0eb50241e35f15d6454e89505, ToolServiceVariableBaseQuery.get omits the configured request headers. The shared implementation affects ToolQuery, ServiceQuery and VariableQuery.

Setting headers through either supported token method, or directly through the public headers mapping, works for the base Query methods but has no effect on the specialized get requests. The request loop supplies only its URL and pagination parameters.

Reproduction

Create any of the three query classes, configure a header, and call get with Requests' send method intercepted. The prepared request does not contain the configured header. This reproduces on every page. No live request is needed.

Expected behavior

Forward the query's headers on every get request, consistent with Query.get, Query.results and Query.hits. Preserve current pagination, output-format handling, empty results and HTTP error propagation.

A focused production fix and 18 offline test cases are prepared. Twelve fail on the original implementation and six controls pass. The complete 147-test suite passes with the fix using the repository-locked HTTP/VCR versions, as do mypy and the repository's gating lint checks. No real credentials or live CMR calls are used.

Lenguaje dominante
Python
Estrellas
29
Forks
35
Sin PR fusionados en 30 d

Preparar el entorno

Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de nasa/python_cmr

Todos los issues de nasa/python_cmr

Issues similares

Más issues de Python

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.