envoyproxy/envoy

Add annotations for certificates listed in /certs

Open

#7604 aperta il 16 lug 2019

Vedi su GitHub
 (6 commenti) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
enhancementhelp wanted

Metriche repository

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

Descrizione

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.

Guida contributor

Direzione di ricerca
Esaminare l'implementazione corrente dell'endpoint /certs in Envoy, comprendere i tipi di certificato (statico, inline, SDS) e determinare dove inserire annotazioni con tipo e nome della risorsa SDS. Guarda la PR #7365 correlata per il formato del dump SecretConfig.
Tech stack
c
Dominio
backendapi
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++Envoy proxy internalsSDS
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
35