envoyproxy/envoy

config validation: validator only checks for file presence not content

Open

#36.289 aperta il 23 set 2024

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

Metriche repository

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

Descrizione

when file system is used to configure Envoy, the config validator (-mode validate) only checks if files are present but does not load those files to check if their content is valid.

In this super simple config:

dynamic_resources:
  cds_config:
    resource_api_version: V3
    path_config_source:
      path: /test/cds.yaml
  lds_config:
    resource_api_version: V3
    path_config_source:
      path: /test/lds.yaml

the validator will accept empty and non-yaml files when run in validate mode.

Guida contributor

Direzione di ricerca
Esamina il flusso di validazione della configurazione nel codice sorgente di Envoy, in particolare come vengono caricati i file durante la validazione. Cerca la funzione che controlla solo l'esistenza del file senza analizzare il contenuto e modificala per analizzare e validare il contenuto del file.
Tech stack
cpp
Dominio
backend
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++Envoy configuration
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60