ThreeMammals/Ocelot

HTTP status 499 seems inappropriate when gateway times out waiting on server

Open

#1 687 ouverte le 29 août 2023

Voir sur GitHub
 (9 commentaires) (5 réactions) (1 assigné)C# (1 617 forks)batch import
bugdocumentationgood first issuehelp wantedproposalquestion

Métriques du dépôt

Stars
 (8 137 stars)
Métriques de merge PR
 (Merge moyen 10j 13h) (4 PRs mergées en 30 j)

Description

Expected Behavior / New Feature

When the gateway (Ocelot) times out waiting for the server to respond, I would expect an HTTP error code of 504, not 499. I wouldn't ever expect a 499 to be returned to the client -- that seems like an error that would just end up in the logs indicating that the client cancelled the request.

Actual Behavior / Motivation for New Feature

With the current behavior, a 499 is returned when Ocelot times out waiting for the server to respond.

Specifications

  • Version: 17.0.0

Guide contributeur

Direction de recherche
Localisez la source du code d'état 499 dans la gestion des délais d'attente d'Ocelot, probablement dans les espaces de noms 'Ocelot.Requester' ou 'Ocelot.Middleware'. Vérifiez les commentaires de l'issue pour tout contexte supplémentaire. La correction consiste à remplacer 499 par 504 dans le chemin de code concerné. Assurez vous que le changement respecte les normes HTTP.
Stack technique
csharprest api
Domaine
backendapi
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Knowledge of HTTP status codesFamiliarity with .NET and Ocelot basics
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60