algolia / algolia/algoliasearch-client-python

runtime error trying to deserialise non json responses

Abierto
#480 0 comentarios 3 reacciones 0 asignados Ver en GitHub
Bug
Lenguaje dominante
Python
Estrellas
207
Forks
63
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

The Python API client throws a runtime exception while attempting to desialize a non json response. Note that, as described on the JavaScript API Client, we should **NOT** deserialise responses on those cases:

```js
isTimedOut || isNetworkError(response) || (~~(status / 100) !== 2 && ~~(status / 100) !== 4)
```

Source: https://github.com/algolia/algoliasearch-client-javascript/blob/master/packages/transporter/src/concerns/retryDecision.ts#L12.

Solution: Create a pull request that ensures that no `response.json()` is performed on the cases mentioned above.

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Comienza en el manejo de respuestas del cliente de la API de Python y compara su comportamiento con la lógica de JavaScript en retryDecision.ts enlazada. Verifica que response.json() se omita para los timeouts, los errores de red y los estados que no sean 2xx ni 4xx; se considera terminado cuando esos casos ya no provocan una excepción de deserialización.

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

Evaluación

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

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.