Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
vvatelot opened this issue Dec 28, 2023 · 2 comments · May be fixed by #322
Open

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

vvatelot opened this issue Dec 28, 2023 · 2 comments · May be fixed by #322
Assignees
Labels
enhancement New feature or request keep open

Comments

@vvatelot
Copy link
Member

static ERROR_MESSAGES = {

@vvatelot vvatelot added the enhancement New feature or request label Dec 28, 2023
@vvatelot vvatelot self-assigned this Dec 28, 2023
@vvatelot vvatelot changed the title Ajouter erreurs 401 et 403 Améliorer affichage sur erreur 521 quand 401 ou 403 Dec 28, 2023
@bgatellier
Copy link
Contributor

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
Copy link
Member Author

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...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request keep open
Projects
None yet
2 participants