cnumr / ecoindex_bff

Simple Back For Front project to interface with Ecoindex API and facilitate Badge status generation
MIT License
2 stars 0 forks source link

[Enhancement]: Améliorer les retours de l'API - Demande de mesure d'une URL #19

Closed hrenaud closed 1 year ago

hrenaud commented 1 year ago

Demande de mesure d'une URL

Pourquoi recevoir un numéro de task (ce devrait être du json), plutôt qu'attendre et retourner l'arbre ?

"35c64f8e-9bca-4c0d-b5db-9dda4d1c6fe5"

Proposition

hrenaud commented 1 year ago

Pourquoi ce choix de retourner une erreur 425 quand on fait /tasks/ecoindexes/{id} et qu'elle est en pending ? Pourquoi ne pas attendre un résultat avant de retourner une erreur ou une mesure ?

{
  "id":"dfab21b1-827d-4259-8ba2-cc51b68847af",
  "status":"PENDING",
  "ecoindex_result":null,
  "task_error":null
}
vvatelot commented 1 year ago

Hello, j'avais répondu à ta question dans ce commentaire: :wink: https://github.com/cnumr/ecoindex_api/issues/309#issuecomment-1479824463

Après, je peux te donner un peu plus d'infos sur ce post linkedin

Voilà le fonctionnement actuel pour expliquer ce qu'il se passe: image

hrenaud commented 1 year ago

dans ton commentaire, tu parlais de la tache https://cors.redoc.ly/v1/tasks/ecoindexes (qui ne s'appelle plus comme ça) là je parle de la suivante, celle où je passe l'id et je demande le résultat. Comme je tombe en erreur, il faut que je lise quand même le résultat pour lire le status et je boucle pour réessayer plus tard. Ce sont des appels reseaux pour rien, ce qui n'est pas recommandé par le greenIT, non ;)

vvatelot commented 1 year ago

425 = Too early

C'est un code standard HTTP: La tâche n'a pas encore été traitée, il faut donc réessayer plus tard (en moyenne, 15sec, mais peut être plus long si bcp de tâches en attentes de traitement)

Tant que la tâche n'est pas traitée, on renvoie ce code erreur et lorsqu'elle a été traitée, on renvoie une 200 avec le résultat

hrenaud commented 1 year ago

ça génère une erreur dans la console du navigateur, c'est dommage.