envoyproxy/envoy

Upstream Access Log can be incorrect if upstream stream outlives downstream

Offen

#26.413 geöffnet am 28.03.2023

Auf GitHub ansehenContributor Guide
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
area/access_logarea/httpbughelp wanted

Repository-Metriken

Stars
 (27.997 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

Access Log: Can be incorrect if upstream stream outlives downstream

Description:

What issue is being seen? Describe what should be happening instead of the bug, for example: Envoy should not crash, the expected value isn't returned, etc.

As demonstrated by https://github.com/envoyproxy/envoy/pull/26196/files#diff-478cdae3dba52e78ed6d1a2e4e4eb4a1aba3e80178b6c8d14707501fc1fb2ddbR3413-R3498

There are some edge scenarios where the upstream access logs will not reflect the actuality of the upstream stream due to when the access logs are captured -- on downstream stream destruction.

Contributor Guide

Research-Richtung
Untersuchen Sie den Zeitpunkt der Zugriffsprotokollerfassung im Verhältnis zur Lebensdauer von Upstream und Downstream Streams. Überprüfen Sie den referenzierten PR (https://github.com/envoyproxy/envoy/pull/26196/files#diff 478cdae3dba52e78ed6d1a2e4e4eb4a1aba3e80178b6c8d14707501fc1fb2ddbR3413 R3498), um den Randfall zu verstehen und einen Fix vorzuschlagen, der sicherstellt, dass Zugriffsprotokolle den tatsächlichen Upstream Stream Status widerspiegeln, selbst wenn der Upstream Stream den Downstream Stream überlebt.
Tech Stack
cpp
Domain
backendinfrastructurenetworking
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.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
C++Envoy internalsaccess logging
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40