envoyproxy/envoy

DNS config: validate mutually exclusive options when loading dns config.

Open

#13.512 aperta il 12 ott 2020

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
area/dnshelp wantedtech debt

Metriche repository

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

Descrizione

#13074 added a config option that is mutually exclusive with the existing (resolvers, using tcp for resolution) dns config options. Currently using these options together results in a release assert being triggered.

This issue tracks adding a centralized config validation function that always runs on the main thread. This way the validation can safely throw an exception. When this validation is added the release assert in createDnsResolver can be deleted.

Guida contributor

Direzione di ricerca
Identifica le opzioni di configurazione mutualmente esclusive e l'assert di rilascio in createDnsResolver. Aggiungi una funzione di validazione che viene eseguita sul thread principale e lancia un'eccezione in caso di conflitto. Rimuovi l'assert.
Tech stack
cpp
Dominio
backendnetworking
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75