envoyproxy/envoy

config validation: validator only checks for file presence not content

Open

#36 289 ouverte le 23 sept. 2024

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)C++ (5 373 forks)batch import
area/config_validationhelp wanted

Métriques du dépôt

Stars
 (27 997 stars)
Métriques de merge PR
 (Merge moyen 8j) (378 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Examinez le flux de validation de la configuration dans le code source d'Envoy, en particulier la manière dont les fichiers sont chargés lors de la validation. Recherchez la fonction qui vérifie uniquement l'existence du fichier sans analyser son contenu et modifiez la pour analyser et valider le contenu du fichier.
Stack technique
cpp
Domaine
backend
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.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++Envoy configuration
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60