algolia / algolia/algoliasearch-client-python

runtime error trying to deserialise non json responses

Ouverte
#480 0 commentaires 3 réactions 0 personnes assignées Voir sur GitHub
Bug
Langage dominant
Python
Étoiles
207
Forks
63
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

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.

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Commencez par la gestion des réponses du client d’API Python et comparez son comportement avec la logique JavaScript liée dans retryDecision.ts. Vérifiez que response.json() est ignoré pour les timeouts, les erreurs réseau et les statuts qui ne sont ni 2xx ni 4xx ; le travail est considéré comme terminé lorsque ces cas ne lèvent plus d’exception de désérialisation.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
python
Domaine
api
Type d'issue
Bug
Difficulté
2/5
Temps estimé
1-3 heures
Activité
À l'abandon
Clarté
Clairement spécifiée
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.