envoyproxy/envoy

network: address interface should pair port:proto

Open

#17.429 geöffnet am 20. Juli 2021

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
area/listenerhelp wantedtech debt

Repository-Metriken

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

Beschreibung

As per the discussion here: https://github.com/envoyproxy/envoy/pull/17414

In a perfect world our Address interface should include proto with port, as the operating system treats TCP:1234 differently from UDP:1234. Unfortunately, this will have very large implications on the overall code structure. There are a few things we might consider here:

  1. Remove port from the primary Address interface, as certain use cases do not actually care about port and proto. For example DNS, IP tagging, etc.
  2. Introduce a AddressAndPort interface which includes an address as well as a port/proto pair. Then, any port must have an associated proto with it as well.

Either way this needs some thinking and prototyping.

Contributor Guide

Research-Richtung
Studieren Sie die aktuelle Address Schnittstelle im Codebase. Überprüfen Sie die verlinkte PR Diskussion (#17414) für den Kontext. Identifizieren Sie alle Verwendungen der Address Schnittstelle und bewerten Sie die Auswirkungen des Hinzufügens von port:proto. Prototypen Sie einen der vorgeschlagenen Ansätze, beginnen Sie mit einer minimalen Änderung, um die Auswirkungen zu verstehen.
Tech Stack
cpp
Domain
infrastructure
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
4
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Über 1 Woche
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++GitComputer networks
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
25