twitter/finagle

ExceptionSourceFilter does not properly attribute CancelledRequestExceptions

Open

#760 ouverte le 13 mars 2019

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)Scala (1 435 forks)batch import
help wanted

Métriques du dépôt

Stars
 (8 864 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

ExceptionSourceFilter does not properly attribute CancelledRequestExceptions

Expected behavior

The source should be attributed to the client (or at least not to the downstream service)

Actual behavior

When a future is cancelled by a client, the source is still attributed to the downstream service. This leads to confusion when the error message is along the lines of:

com.twitter.finagle.CancelledRequestException: request cancelled. Remote Info: Upstream Address: Not Available, Upstream id: Not Available, Downstream Address: my.downstream.host/X.X.X.X:443, Downstream label: my.downstream.host:443, Trace Id: efdec7e41b7c30fa.d535d24f770f3fa8<:efdec7e41b7c30fa

even though the downstream was not involved in the request at all.

Steps to reproduce the behavior

Cancel a future and observe the attribution from ExceptionSourceFilter.

Guide contributeur

Direction de recherche
Étudiez l'implémentation d'ExceptionSourceFilter dans la base de code de Finagle pour comprendre comment elle attribue les exceptions à des sources. Plus précisément, examinez comment les CancelledRequestExceptions sont gérées lors de l'annulation par le client. Observez le flux où un future est annulé par un client et tracez la logique d'attribution des exceptions. Consultez le commentaire existant sur l'issue pour un contexte supplémentaire. Envisagez de modifier le filtre pour attribuer correctement la source au client ou la définir sur 'Non disponible' lorsqu'une annulation se produit.
Stack technique
scala
Domaine
backend
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-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Scala basicsFamiliarity with Finagle
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
45