envoyproxy / envoyproxy/envoy

Restrict visibility to Envoy internals in consuming projects

Aperta
#827 2 commenti 0 reazioni 0 assegnatari Vedi su GitHub
area/build help wanted tech debt
Lingua principale
C++
Stelle
28.9k
Fork
5.6k
Merge medio
1g 22h
PR unite (30g)
430

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 per i contributori

Apri la guida per i contributori

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.