envoyproxy/envoy

config validation: validator only checks for file presence not content

Open

#36.289 aberto em 23 de set. de 2024

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)C++ (5.373 forks)batch import
area/config_validationhelp wanted

Métricas do repositório

Stars
 (27.997 stars)
Métricas de merge de PR
 (Mesclagem média 8d) (378 fundiu PRs em 30d)

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.

Guia do colaborador

Direção de pesquisa
Examine o fluxo de validação de configuração no código fonte do Envoy, particularmente como os arquivos são carregados durante a validação. Procure a função que verifica apenas a existência do arquivo sem analisar o conteúdo e modifique a para analisar e validar o conteúdo do arquivo.
Pilha de tecnologia
cpp
Domain
backend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++Envoy configuration
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60