envoyproxy/envoy

Upstream Access Log can be incorrect if upstream stream outlives downstream

Ouverte

#26 413 ouverte le 28 mars 2023

Voir sur GitHubGuide contributeur
 (3 commentaires) (0 réaction) (0 personne assignée)C++ (5 373 forks)batch import
area/access_logarea/httpbughelp wanted

Métriques du dépôt

Stars
 (27 997 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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.

Guide contributeur

Direction de recherche
Examiner le moment de la capture du journal d'accès par rapport à la durée de vie des flux en amont et en aval. Consultez la PR référencée (https://github.com/envoyproxy/envoy/pull/26196/files#diff 478cdae3dba52e78ed6d1a2e4e4eb4a1aba3e80178b6c8d14707501fc1fb2ddbR3413 R3498) pour comprendre le cas limite et proposer un correctif garantissant que les journaux d'accès reflètent l'état réel du flux en amont même lorsque le flux en amont survit au flux en aval.
Stack technique
cpp
Domaine
backendinfrastructurenetworking
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.
Une demi journée
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.
Plutôt claire
Prérequis
C++Envoy internalsaccess logging
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40