envoyproxy/envoy

Add annotations for certificates listed in /certs

Open

#7.604 aberto em 16 de jul. de 2019

Ver no GitHub
 (6 comments) (0 reactions) (0 assignees)C++ (5.373 forks)batch import
enhancementhelp 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

Title: Add annotations for certificates listed in /certs

Description: Per discussion with @PiotrSikora and @incfly on https://github.com/envoyproxy/envoy/issues/7111#issuecomment-499291659, we need to add annotations or something similar into /certs dump to clearly display the type of each certificate, i.e., certificates from static secret, inline secret or SDS. It would be nice if we can add SDS resource name and other information into the annotation, so that we can associate certificates from /certs to SecretConfig(PR https://github.com/envoyproxy/envoy/pull/7365) dump from /config_dump while debugging SDS related issues.

Guia do colaborador

Direção de pesquisa
Examine a implementação atual do endpoint /certs no Envoy, entenda os tipos de certificado (estático, inline, SDS) e determine onde injetar anotações com tipo e nome do recurso SDS. Veja o PR #7365 relacionado para o formato do dump SecretConfig.
Pilha de tecnologia
c
Domain
backendapi
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
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 proxy internalsSDS
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
35