Skip to content
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

build: disable guard dog #2582

Closed
wants to merge 1 commit into from
Closed

build: disable guard dog #2582

wants to merge 1 commit into from

Conversation

jpsim
Copy link
Contributor

@jpsim jpsim commented Sep 29, 2022

Commit Message: build: disable guard dog
Additional Description: Which Envoy Mobile should never enable, because terminating the Envoy process would mean terminating the host iOS or Android app. See #2581 for more details.
Risk Level: Low, compile-time flag, no-op by default.
Testing: Building Envoy Mobile locally.
Docs Changes: None.
Release Notes: None.
Platform Specific Features: Will be disabled when building Envoy Mobile.
Fixes: #2581

Signed-off-by: JP Simard [email protected]

Signed-off-by: JP Simard <[email protected]>
@jpsim jpsim changed the title Disable guard dog build: disable guard dog Sep 29, 2022
@jpsim
Copy link
Contributor Author

jpsim commented Dec 1, 2022

Closing as stale now that we've migrated to the Envoy repo: https://github.com/envoyproxy/envoy/blob/main/mobile/README.md

@jpsim jpsim closed this Dec 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Disable GuardDogImpl
1 participant