[redis_proxy] added a constraint for route.prefix().size() (#12637)

Signed-off-by: jianwen <jianwendong@google.com>
envoyproxy · Aug 14, 2020 · 7852b9d · 7852b9d
1 parent cf9b494
commit 7852b9d
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 2 deletions.
diff --git a/api/envoy/extensions/filters/network/redis_proxy/v3/redis_proxy.proto b/api/envoy/extensions/filters/network/redis_proxy/v3/redis_proxy.proto
@@ -156,7 +156,7 @@ message RedisProxy {
 
       // String prefix that must match the beginning of the keys. Envoy will always favor the
       // longest match.
-      string prefix = 1;
+      string prefix = 1 [(validate.rules).string = {max_bytes: 1000}];
 
       // Indicates if the prefix needs to be removed from the key when forwarded.
       bool remove_prefix = 2;

diff --git a/generated_api_shadow/envoy/extensions/filters/network/redis_proxy/v3/redis_proxy.proto b/generated_api_shadow/envoy/extensions/filters/network/redis_proxy/v3/redis_proxy.proto
diff --git a/...ommon/fuzz/network_readfilter_corpus/oss_redis_proxy_stackoverflow_with_long_route_prefix b/...ommon/fuzz/network_readfilter_corpus/oss_redis_proxy_stackoverflow_with_long_route_prefix