Skip to content

Commit

Permalink
Update Envoy (#2669)
Browse files Browse the repository at this point in the history
* Update Envoy
* Remove uses of c-ares
* Register static cluster extension
* Fix KV store APIs

Co-authored-by: Alyssa Wilk <[email protected]>
Signed-off-by: JP Simard <[email protected]>
  • Loading branch information
2 people authored and jpsim committed Nov 28, 2022
1 parent 6245dbf commit bf327c0
Show file tree
Hide file tree
Showing 6 changed files with 8 additions and 6 deletions.
2 changes: 1 addition & 1 deletion mobile/envoy
Submodule envoy updated from e4ae22 to e711d1
3 changes: 2 additions & 1 deletion mobile/envoy_build_config/BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,9 @@ envoy_cc_library(
"extension_registry_platform_additions",
"@envoy//source/common/network:socket_lib",
"@envoy//source/common/router:upstream_codec_filter_lib",
"@envoy//source/extensions/clusters/logical_dns:logical_dns_cluster_lib",
"@envoy//source/extensions/clusters/dynamic_forward_proxy:cluster",
"@envoy//source/extensions/clusters/logical_dns:logical_dns_cluster_lib",
"@envoy//source/extensions/clusters/static:static_cluster_lib",
"@envoy//source/extensions/compression/brotli/decompressor:config",
"@envoy//source/extensions/compression/gzip/decompressor:config",
"@envoy//source/extensions/filters/http/alternate_protocols_cache:config",
Expand Down
2 changes: 2 additions & 0 deletions mobile/envoy_build_config/extension_registry.cc
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
#include "source/common/router/upstream_codec_filter.h"
#include "source/extensions/clusters/dynamic_forward_proxy/cluster.h"
#include "source/extensions/clusters/logical_dns/logical_dns_cluster.h"
#include "source/extensions/clusters/static/static_cluster.h"
#include "source/extensions/compression/brotli/decompressor/config.h"
#include "source/extensions/compression/gzip/decompressor/config.h"
#include "source/extensions/filters/http/alternate_protocols_cache/config.h"
Expand Down Expand Up @@ -35,6 +36,7 @@
namespace Envoy {

void ExtensionRegistry::registerFactories() {
Envoy::Upstream::forceRegisterStaticClusterFactory();
Envoy::Extensions::Clusters::DynamicForwardProxy::forceRegisterClusterFactory();
Envoy::Extensions::Compression::Brotli::Decompressor::
forceRegisterBrotliDecompressorLibraryFactory();
Expand Down
1 change: 1 addition & 0 deletions mobile/envoy_build_config/extensions_build_config.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ EXTENSION_PACKAGE_VISIBILITY = ["//visibility:public"]
EXTENSIONS = {
"envoy.clusters.dynamic_forward_proxy": "//source/extensions/clusters/dynamic_forward_proxy:cluster",
"envoy.clusters.logical_dns": "//source/extensions/clusters/logical_dns:logical_dns_cluster_lib",
"envoy.clusters.static": "//source/extensions/clusters/static:static_cluster_lib",
"envoy.filters.connection_pools.http.generic": "//source/extensions/upstreams/http/generic:config",
"envoy.filters.http.alternate_protocols_cache": "//source/extensions/filters/http/alternate_protocols_cache:config",
"envoy.filters.http.assertion": "@envoy_mobile//library/common/extensions/filters/http/assertion:config",
Expand Down
4 changes: 2 additions & 2 deletions mobile/library/common/extensions/key_value/platform/config.cc
Original file line number Diff line number Diff line change
Expand Up @@ -62,8 +62,8 @@ PlatformKeyValueStore::PlatformKeyValueStore(Event::Dispatcher& dispatcher,

void PlatformKeyValueStore::flush() {
std::string output;
for (const auto& [key, value] : store_) {
std::string string_value = value.first;
for (const auto& [key, value_with_ttl] : store()) {
std::string string_value = value_with_ttl.value_;
absl::StrAppend(&output, key.length(), "\n", key, string_value.length(), "\n", string_value);
}
platform_interface_.save(key_, output);
Expand Down
2 changes: 0 additions & 2 deletions mobile/library/common/jni/jni_interface.cc
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
#include <ares.h>

#include "library/common/api/c_types.h"
#include "library/common/data/utility.h"
#include "library/common/extensions/filters/http/platform_bridge/c_types.h"
Expand Down

0 comments on commit bf327c0

Please sign in to comment.