twitter/finagle

ExceptionSourceFilter does not properly attribute CancelledRequestExceptions

Open

#760 geöffnet am 13. März 2019

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)Scala (1.435 Forks)batch import
help wanted

Repository-Metriken

Stars
 (8.864 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Untersuchen Sie die Implementierung von ExceptionSourceFilter in der Finagle Codebasis, um zu verstehen, wie Ausnahmen Quellen zugeordnet werden. Untersuchen Sie insbesondere, wie CancelledRequestExceptions während der Client Stornierung behandelt werden. Betrachten Sie den Ablauf, bei dem ein Future von einem Client abgebrochen wird, und verfolgen Sie die Logik zur Ausnahmezuordnung. Überprüfen Sie den vorhandenen Kommentar zum Issue für zusätzlichen Kontext. Erwägen Sie, den Filter so zu ändern, dass die Quelle bei einer Stornierung korrekt dem Client zugeordnet oder auf 'Nicht verfügbar' gesetzt wird.
Tech Stack
scala
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Scala basicsFamiliarity with Finagle
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
45