cnumr/EcoIndex

[Bug]: Pas de gestion des timeouts de l'API

bgatellier opened this issue · 3 comments

Que s'est il passé ?

J'ai lancé l'analyse de l'URL https://www.mk2.com
La popup d'analyse est bien affichée avec le loader.
La requête vers l'API (POST https://ecoindex.p.rapidapi.com/v1/ecoindexes) échoue via un timeout, qui n'est pas géré (erreur JavaScript dans la console et aucun changement dans l'interface, voir capture d'écran ci-dessous)

image

Sur quel navigateur avez vous rencontré ce problème ?

Firefox, Chrome, Microsoft Edge

Sur quel device rencontrez vous le problème ?

PC

Un message d'erreur est il apparu ?

Uncaught (in promise) TimeoutError: Request timed out
    p https://www.ecoindex.fr/fr/js/bundle.b25f99fdb63e49829051b98356c2cbafb2daed4b2d44aef15783f05bf2c8e0c2.js:1
    i https://www.ecoindex.fr/fr/js/bundle.b25f99fdb63e49829051b98356c2cbafb2daed4b2d44aef15783f05bf2c8e0c2.js:1
    setTimeout handler*L/< https://www.ecoindex.fr/fr/js/bundle.b25f99fdb63e49829051b98356c2cbafb2daed4b2d44aef15783f05bf2c8e0c2.js:1
    L https://www.ecoindex.fr/fr/js/bundle.b25f99fdb63e49829051b98356c2cbafb2daed4b2d44aef15783f05bf2c8e0c2.js:1
    _fetch https://www.ecoindex.fr/fr/js/bundle.b25f99fdb63e49829051b98356c2cbafb2daed4b2d44aef15783f05bf2c8e0c2.js:1
    o https://www.ecoindex.fr/fr/js/bundle.b25f99fdb63e49829051b98356c2cbafb2daed4b2d44aef15783f05bf2c8e0c2.js:1
    create https://www.ecoindex.fr/fr/js/bundle.b25f99fdb63e49829051b98356c2cbafb2daed4b2d44aef15783f05bf2c8e0c2.js:1
    t https://www.ecoindex.fr/fr/js/bundle.b25f99fdb63e49829051b98356c2cbafb2daed4b2d44aef15783f05bf2c8e0c2.js:1
    #e https://www.ecoindex.fr/fr/js/bundle.b25f99fdb63e49829051b98356c2cbafb2daed4b2d44aef15783f05bf2c8e0c2.js:1
    newAnalysisByURL https://www.ecoindex.fr/fr/js/bundle.b25f99fdb63e49829051b98356c2cbafb2daed4b2d44aef15783f05bf2c8e0c2.js:1
    launchAnalysisByURL https://www.ecoindex.fr/fr/js/bundle.b25f99fdb63e49829051b98356c2cbafb2daed4b2d44aef15783f05bf2c8e0c2.js:10
    Pt https://www.ecoindex.fr/fr/js/bundle.b25f99fdb63e49829051b98356c2cbafb2daed4b2d44aef15783f05bf2c8e0c2.js:10
    kt https://www.ecoindex.fr/fr/js/bundle.b25f99fdb63e49829051b98356c2cbafb2daed4b2d44aef15783f05bf2c8e0c2.js:10
    kt https://www.ecoindex.fr/fr/js/bundle.b25f99fdb63e49829051b98356c2cbafb2daed4b2d44aef15783f05bf2c8e0c2.js:10
    St https://www.ecoindex.fr/fr/js/bundle.b25f99fdb63e49829051b98356c2cbafb2daed4b2d44aef15783f05bf2c8e0c2.js:10
    EventListener.handleEvent* https://www.ecoindex.fr/fr/js/bundle.b25f99fdb63e49829051b98356c2cbafb2daed4b2d44aef15783f05bf2c8e0c2.js:10
    <anonymous> https://www.ecoindex.fr/fr/js/bundle.b25f99fdb63e49829051b98356c2cbafb2daed4b2d44aef15783f05bf2c8e0c2.js:10
bundle.b25f99fdb63e49829051b98356c2cbafb2daed4b2d44aef15783f05bf2c8e0c2.js:1:606
    #t https://www.ecoindex.fr/fr/js/bundle.b25f99fdb63e49829051b98356c2cbafb2daed4b2d44aef15783f05bf2c8e0c2.js:10
    launchAnalysisByURL https://www.ecoindex.fr/fr/js/bundle.b25f99fdb63e49829051b98356c2cbafb2daed4b2d44aef15783f05bf2c8e0c2.js:10
    AsyncFunctionThrow self-hosted:811
    (Asynchrone : async)
    Pt https://www.ecoindex.fr/fr/js/bundle.b25f99fdb63e49829051b98356c2cbafb2daed4b2d44aef15783f05bf2c8e0c2.js:10
    kt https://www.ecoindex.fr/fr/js/bundle.b25f99fdb63e49829051b98356c2cbafb2daed4b2d44aef15783f05bf2c8e0c2.js:10
    (Asynchrone : EventListener.handleEvent)
    kt https://www.ecoindex.fr/fr/js/bundle.b25f99fdb63e49829051b98356c2cbafb2daed4b2d44aef15783f05bf2c8e0c2.js:10
    St https://www.ecoindex.fr/fr/js/bundle.b25f99fdb63e49829051b98356c2cbafb2daed4b2d44aef15783f05bf2c8e0c2.js:10
    (Asynchrone : EventListener.handleEvent)
    <anonyme> https://www.ecoindex.fr/fr/js/bundle.b25f99fdb63e49829051b98356c2cbafb2daed4b2d44aef15783f05bf2c8e0c2.js:10
    <anonyme> https://www.ecoindex.fr/fr/js/bundle.b25f99fdb63e49829051b98356c2cbafb2daed4b2d44aef15783f05bf2c8e0c2.js:10

🙏🏻 Praise: merci pour le rapport d'incident très complet ! On travaille en parallèle sur un autre mode de fonctionnement donc à voir comment on gère ce bug...

#180

yaaax commented

Merci @bgatellier pour ce bug.
#185 devrait corriger ça.

yaaax commented

Fixed by #185