envoyproxy/envoy

Fine grained logging via FANCY_LOG always evaluates macro arguments

Open

#12.885 aperta il 29 ago 2020

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
area/envoy_loghelp wanted

Metriche repository

Star
 (27.997 star)
Metriche merge PR
 (Merge medio 8g) (378 PR mergiate in 30 g)

Descrizione

The ENVOY_LOG macro without fine grained logging doesn't evaluate variadic arguments when the log level squelches the message. When going through FANCY_LOG however, that no longer holds. Variadic arguments are always evaluated regardless of the log level involved. This could be problematic from a perf point of view.

Context: https://github.com/envoyproxy/envoy/pull/12830#issuecomment-681918730

Guida contributor

Direzione di ricerca
Ispeziona la definizione della macro FANCY LOG e traccia il percorso di valutazione degli argomenti variadici per trovare dove avviene la valutazione anticipata.
Tech stack
cpp
Dominio
backendinfrastructure
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++Macros
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50