-
Notifications
You must be signed in to change notification settings - Fork 84
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
example apps: disable interface binding #2456
Conversation
Signed-off-by: Rafal Augustyniak <[email protected]>
There is a crash that I saw when running an example app with interface binding and proxy code from #2416 enabled. Crash happens because an instance of envoy-mobile/library/common/network/connectivity_manager.cc Lines 356 to 388 in 308f15c
Full crash stacktrace:
|
Re. crash mentioned above - it turns out that it's not interface binding fault that the app was crashing for me. More about this in #2457. TL;DR is that I was compiling the app using |
Description: After finding an interface binding related crash in #2456 I noticed that our `dbg` bazel config does not specify `--define=include_ifaddrs=true` which is supposed to ensure that we compile code that's required for proper functioning of Envoy Mobile on Android. Introduce `dbg-android` and `dbg-ios` which are intended to be platform specific `dbg` configurations. Risk Level: None, does not impact release configurations. Testing: Ran Kotlin example app manually. Docs Changes: N/A Release Notes: N/A Signed-off-by: Rafal Augustyniak <[email protected]>
Description: After finding an interface binding related crash in envoyproxy/envoy-mobile#2456 I noticed that our `dbg` bazel config does not specify `--define=include_ifaddrs=true` which is supposed to ensure that we compile code that's required for proper functioning of Envoy Mobile on Android. Introduce `dbg-android` and `dbg-ios` which are intended to be platform specific `dbg` configurations. Risk Level: None, does not impact release configurations. Testing: Ran Kotlin example app manually. Docs Changes: N/A Release Notes: N/A Signed-off-by: Rafal Augustyniak <[email protected]> Signed-off-by: JP Simard <[email protected]>
Description: Disable interface binding in Android Kotlin example app as we have no clear plans for enabling interface binding anytime soon and having it disabled makes debugging of the example app (and Envoy Mobile) easier. On top of that, the same feature is not enabled in iOS example apps.
Risk Level: None, example app change only.
Testing: Example app run manually.
Docs Changes: N/A
Release Notes: N/A
Signed-off-by: Rafal Augustyniak [email protected]