cnumr/EcoIndex

Améliorer affichage sur erreur 521 quand 401 ou 403

vvatelot opened this issue · 2 comments

vvatelot commented

EcoIndex/assets/js/components/EcoIndexDialog.js

Line 11 in 73797d0

static ERROR_MESSAGES = {

bgatellier commented

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

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...
  • 👍1