envoyproxy/envoy

config validation: validator only checks for file presence not content

Open

#36.289 geöffnet am 23. Sept. 2024

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
area/config_validationhelp wanted

Repository-Metriken

Stars
 (27.997 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 8T) (378 gemergte PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Untersuche den Konfigurationsvalidierungsablauf im Envoy Quellcode, insbesondere wie Dateien während der Validierung geladen werden. Suche nach der Funktion, die nur die Existenz der Datei überprüft, ohne den Inhalt zu parsen, und ändere sie, um den Dateiinhalt zu parsen und zu validieren.
Tech Stack
cpp
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++Envoy configuration
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60