envoyproxy / envoy-mobile

Client HTTP and networking library based on the Envoy project for iOS, Android, and more.
https://envoymobile.io
Apache License 2.0
560 stars 84 forks source link

remove downstream listeners from Envoy Mobile build #2711

Closed alyssawilk closed 1 year ago

alyssawilk commented 1 year ago

based on a hacked up prototype looks like ~1M savings

alyssawilk commented 1 year ago

@jpsim next step is y'all experimenting with the use_api_listener runtime guard.

jpsim commented 1 year ago

That'd be setRuntimeGuard("use_api_listener", true / false) right? Is there a way we can observe if it's being applied correctly?

jpsim commented 1 year ago

I'm comfortable removing the listeners completely. It looks like all our unit and integration tests pass with listeners disabled and I don't think we need to run a production experiment for this.