envoyproxy/envoy

network: address interface should pair port:proto

Open

#17.429 aperta il 20 lug 2021

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

Metriche repository

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

Descrizione

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.

Guida contributor

Direzione di ricerca
Studia l'attuale interfaccia Address nel codice. Rivedi la discussione del PR collegato (#17414) per contesto. Identifica tutti gli usi dell'interfaccia Address e valuta l'impatto dell'aggiunta di port:proto. Prototipa uno degli approcci proposti, iniziando con una modifica minima per comprendere le implicazioni.
Tech stack
cpp
Dominio
infrastructure
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
4
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Oltre 1 settimana
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++GitComputer networks
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
25