Améliorer affichage sur erreur 521 quand 401 ou 403

vvatelot commented 7 months ago

https://github.com/cnumr/EcoIndex/blob/73797d00cf0b55dd50d405e7301cabf32995af7d/assets/js/components/EcoIndexDialog.js#L11

bgatellier commented 6 months ago

Hello @vvatelot ,

Je suis pas sûr de bien comprendre l'évolution ici : il faut afficher un message d'erreur spécifique lorsque l'API répond 401 ou 403, c'est bien ça ?

vvatelot commented 5 months ago

En fait, l'explication complète c'est que l'API renvoit un code 521 (code non standard qui décrit qu'il y a eu un problème côté serveur lors de l'analyse) et dans la réponse, on a le détail avec le code réellement renvoyé par la page analysée. Exemple:

GET https://api.ecoindex.fr/v1/tasks/ecoindexes/19f0efc4-a54c-4f48-921a-54acbd6f7225

{
    "id": "19f0efc4-a54c-4f48-921a-54acbd6f7225",
    "status": "SUCCESS",
    "ecoindex_result": {
        "status": "FAILURE",
        "detail": null,
        "error": {
            "detail": {
                "status": 403
            },
            "exception": "EcoindexStatusError",
            "message": "",
            "status_code": 521,
            "url": "https://www.manomano.fr/"
        }
    },
    "task_error": null
}

Ce qui se traduit comme ça:

La tâche d'analyse a tourné correctement
Mais durant celle ci, l'analyse ecoindex a échoué (code 521)
La page analysée a renvoyé une erreur de type 403
Ce qui veut dire que la page est très probablement protégée par une sécurité anti bot...

cnumr / EcoIndex

Améliorer affichage sur erreur 521 quand 401 ou 403 #301