search

romulusFR / lifap5-backend-2019-2020

Backend pour le projet 2019-2020 de LIFAP5 en L2 informatique UCBL
https://perso.liris.cnrs.fr/romuald.thion/dokuwiki/doku.php?id=enseignement:lifap5:start
Other
1 stars 3 forks source link

Erreur 503 du rate limiter, mauvais code ? #21

Open Lifeismana opened 4 years ago

Lifeismana commented 4 years ago

Quand se fait rate limit, on a une 503 du serveur qui correspond à "Service Unavailable" Mais est-ce que ça serait pas plus cohérent de renvoyer une erreur 429 Too Many Requests à la place de la 503 ? Une recherche rapide donne comme solution : 

limit_req_status 429;
limit_conn_status 429;
Lifeismana commented 4 years ago

En plus de ça on pourrait renvoyé un Retry-After avec un peut d'aléatoire pour étaler les requêtes qu'on réessaye et ça résoudrait un peu #17

Webcretaire commented 4 years ago

Je suis d'accord que le code 429 est bien adapté, par contre je ne pense pas que le Retry-After soit une bonne idée, pour la même raison que j'ai expliquée dans #17 : vous encourager à mettre des délais artificiels dans le client pour contrer des défauts du backend est une mauvaise idée, surtout pour un projet scolaire dans lequel on essaye de vous encourager à faire du code propre, fonctionnel, etc.