envoyproxy/envoy

Restrict visibility to Envoy internals in consuming projects

Open

#827 aperta il 24 apr 2017

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

Metriche repository

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

Descrizione

Today, the Bazel visibility for all targets (from source, to test support to the tests themselves) is public. This means that a project that consumes Envoy and adds its own filters etc., might rely on internal implementation details that are not intended to be stable aspects of the API.

We should restrict visibility outside of Envoy to only its interfaces, well defined library/binary exports (//source/exe:envoy, //source/exe:envoy_main_lib) and common code under ::Api:: that should be explicitly shared and is subject to the same API considerations as the Envoy interfaces in include.

Guida contributor

Direzione di ricerca
Esamina le attuali regole di visibilità Bazel per tutti i target nel repository. Identifica quali target sono interni e dovrebbero avere visibilità ristretta. Aggiorna i file BUILD di conseguenza per limitare la visibilità ai target pubblici specificati.
Tech stack
cpp
Dominio
build systeminfrastructure
Tipo issue
Manutenzione
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
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
BazelC++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60