envoyproxy/envoy

Align Windows getaddrinfo behavior to other platforms

Open

#24 326 ouverte le 2 déc. 2022

Voir sur GitHub
 (9 commentaires) (0 réactions) (1 assigné)C++ (5 373 forks)batch import
area/windowsbughelp wantedno stalebot

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

Title: Align Windows getaddrinfo behavior to other platforms

Description: #23957 shows how Windows parsing is more relaxed compared to other platforms. We may want to align it to have the same strictness in order to have a consistent behavior.

Repro steps: Remove the compiler ifndef directive introduce in #23957 and run: ./ci/run_envoy_docker.sh './ci/do_ci.sh bazel.dev //test/common/network:utility_test --test_output=all'

Guide contributeur

Direction de recherche
Examinez la directive ifndef introduite dans le PR #23957 et comprenez comment l'analyse getaddrinfo de Windows diffère des autres plateformes. Testez en supprimant la directive et en exécutant le test bazel spécifié pour voir les échecs.
Stack technique
cpp
Domaine
backend
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.
3
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.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++Bazel
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40