envoyproxy/envoy

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

Open

#13 512 ouverte le 12 oct. 2020

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)C++ (5 373 forks)batch import
area/dnshelp wantedtech debt

Métriques du dépôt

Stars
 (27 997 stars)
Métriques de merge PR
 (Merge moyen 8j) (378 PRs mergées en 30 j)

Description

#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.

Guide contributeur

Direction de recherche
Identifiez les options de configuration mutuellement exclusives et l'assert de sortie dans createDnsResolver. Ajoutez une fonction de validation qui s'exécute sur le thread principal et lance une exception en cas de conflit. Supprimez l'assert.
Stack technique
cpp
Domaine
backendnetworking
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75