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.

Sign up for GitHub

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

Jump to bottom

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

Merged
merged 14 commits into from
Aug 14, 2020
Merged

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

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
00e5deb
added a test case with long string for trie, and added destructor of the
jianwen612 Aug 4, 2020
dcd8885
set string to be const.
jianwen612 Aug 4, 2020
1065f8f
added a comment for destructor
jianwen612 Aug 5, 2020
28ff90b
added a helper function to makecode clearer.
jianwen612 Aug 5, 2020
99f9958
refined the comments
jianwen612 Aug 5, 2020
c7738a1
refined the comments
jianwen612 Aug 5, 2020
b5fcd4c
added regression test for fuzzer
jianwen612 Aug 5, 2020
15bc3ad
Merge remote-tracking branch 'upstream/master' into fix_issue_24558
jianwen612 Aug 6, 2020
4962561
fixed issue 24558 by constrain the length of route.prefix.length()
jianwen612 Aug 13, 2020
3d3022c
restore a empty line
jianwen612 Aug 13, 2020
f777601
removed an empty line
jianwen612 Aug 13, 2020
686548c
added a new line
jianwen612 Aug 13, 2020
3db2a65
Merge remote-tracking branch 'upstream/master' into another_fix_issue…
jianwen612 Aug 13, 2020
3429c3a
add PGV for route.length()
jianwen612 Aug 13, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion api/envoy/extensions/filters/network/redis_proxy/v3/redis_proxy.proto
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down
2 changes: 1 addition & 1 deletion generated_api_shadow/envoy/extensions/filters/network/redis_proxy/v3/redis_proxy.proto
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

11 changes: 11 additions & 0 deletions ...ommon/fuzz/network_readfilter_corpus/oss_redis_proxy_stackoverflow_with_long_route_prefix
View file
Open in desktop

Large diffs are not rendered by default.