envoyproxy/envoy

Align Windows getaddrinfo behavior to other platforms

Open

#24.326 geöffnet am 2. Dez. 2022

Auf GitHub ansehen
 (9 Kommentare) (0 Reaktionen) (1 zugewiesene Person)C++ (5.373 Forks)batch import
area/windowsbughelp wantedno stalebot

Repository-Metriken

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

Beschreibung

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'

Contributor Guide

Research-Richtung
Untersuchen Sie die in PR #23957 eingeführte ifndef Direktive und verstehen Sie, wie sich das Windows getaddrinfo Parsing von anderen Plattformen unterscheidet. Testen Sie, indem Sie die Direktive entfernen und den angegebenen Bazel Test ausführen, um Fehler zu sehen.
Tech Stack
cpp
Domain
backend
Issue Type
Fehler
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-3 Stunden
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++Bazel
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40