envoyproxy/envoy

Access log status code filter not working for response code 0

Open

#17.482 aperta il 26 lug 2021

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

Metriche repository

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

Descrizione

Description: Unable to use status code filters to log requests with a 0 response code.

Repro steps: We are running Istio 1.6.13 which uses Envoy 1.15.4. We want to only output access logs for HTTP requests with responses >= 400 or == 0.

This is the filter being applied:

filter:
  or_filter:
    filters:
    - status_code_filter:
        comparison:
          op: GE
          value:
            default_value: "400"
            runtime_key: "400"
    - status_code_filter:
        comparison:
          op: EQ
          value:
            default_value: "0"
            runtime_key: "0"

All 4xx/5xx requests are being logged, and 1xx/2xx/3xx requests are not logged, as expected. However, any requests with response code 0 are also not being logged.

Has this been resolved in a newer version of Envoy/Istio?

If not:

  1. Is this the intended behaviour because the status code filter only gets applied to valid HTTP response codes?
  2. Is there a different filter that can be applied to make sure all 0 responses get logged?

Guida contributor

Direzione di ricerca
Indaga l'implementazione di status code filter nella catena di filtri HTTP di Envoy. Il filtro probabilmente elabora solo i codici di stato HTTP validi (100-599) e ignora lo 0. Esamina la logica di confronto del filtro in source/common/http/filters. Verifica se esiste una limitazione nota o se è necessario un caso speciale per lo 0. Inoltre, rivedi il parsing della configurazione per vedere se lo 0 è accettato come valore valido. Considera di aggiungere un caso di test per il codice di stato 0.
Tech stack
cpp
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65