envoyproxy/envoy

Add annotations for certificates listed in /certs

Open

#7.604 geöffnet am 16. Juli 2019

Auf GitHub ansehen
 (6 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
enhancementhelp wanted

Repository-Metriken

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

Beschreibung

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.

Contributor Guide

Research-Richtung
Untersuchen Sie die aktuelle Implementierung des /certs Endpunkts in Envoy, verstehen Sie die Zertifikatstypen (statisch, inline, SDS) und bestimmen Sie, wo Annotationen mit Typ und SDS Ressourcennamen eingefügt werden sollen. Sehen Sie sich den zugehörigen PR #7365 für das SecretConfig Dump Format an.
Tech Stack
c
Domain
backendapi
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++Envoy proxy internalsSDS
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
35