envoyproxy/envoy

Add verify_client_certificate.

Open

#5501 aperta il 6 gen 2019

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
area/tlshelp wanted

Metriche repository

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

Descrizione

From https://github.com/envoyproxy/envoy/pull/5207#issuecomment-447345279:

I think that we could simply add verify_client_certificate: true|check|false, where:

  • true would verify the client certificate and reject connection if the verification failed (what we have today),
  • check would verify the client certificate, but forward it and the verification status via x-forwarded-client-cert or another header (for easier matching), regardless of the verification status,
  • false would forward the client certificate to the backend via x-forwarded-client-cert without attempting to verify it (to avoid crypto operations, do access control at the backend, etc.).

Guida contributor

Direzione di ricerca
Esamina il codice attuale di verifica del certificato client TLS in Envoy, in particolare nella gestione della connessione SSL. Identifica dove è attualmente implementato verify client certificate (probabilmente solo true/false) e pianifica come aggiungere una modalità 'check' che inoltra lo stato di verifica.
Tech stack
c
Dominio
security
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.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
C++TLS
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
30