-
Notifications
You must be signed in to change notification settings - Fork 5.5k
http: support creating filters with match tree #14430
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
Merged
Merged
Changes from 35 commits
Commits
Show all changes
37 commits
Select commit
Hold shift + click to select a range
ecf47e2
http: add support for skip filter match action
910d3f1
support skipping filter execution
5734a5c
move protos
e63ff82
Merge branch 'master' of github.com:envoyproxy/envoy into hcm-int
ce3e503
move protos
6b407f5
spelling + title
6fec69f
add to toc
baff1bd
templating to help clang do the conversions
c39c870
fix name collision
571f096
update mocks
e83b3b2
remove accidental line
9afefee
wip filter for creating match trees
3d223ad
wire things up a bit
d124637
Merge remote-tracking branch 'envoy/master' into matching-filter
ecb49e7
add test coverage
13a552d
remove old protos
a66c51f
cleanup
b66ef32
more cleanups
01fced2
format
142c70b
remove more old protos
9ce8523
add comments
be5f50b
test coverage
711114b
update comments
d8e5cde
use better helpers
465f9a4
comments + format
e88be1d
move protos out of hcm config
487822f
move to envoy/type/matcher
6cb618f
update docs
2cfb10c
format
0fb52fb
fix test failures
91e62af
fix last type url
b2c884f
clang-tidy
e4987b5
add include
8868cd2
Merge remote-tracking branch 'envoy/master' into matching-filter
9abac4b
Merge remote-tracking branch 'envoy/master' into matching-filter
899a365
introduce templated base class for factory
193a95c
Merge remote-tracking branch 'envoy/master' into matching-filter
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,35 @@ | ||
| syntax = "proto3"; | ||
|
|
||
| package envoy.type.matcher.v3; | ||
|
|
||
| import "udpa/annotations/migrate.proto"; | ||
| import "udpa/annotations/status.proto"; | ||
| import "udpa/annotations/versioning.proto"; | ||
| import "validate/validate.proto"; | ||
|
|
||
| option java_package = "io.envoyproxy.envoy.type.matcher.v3"; | ||
| option java_outer_classname = "HttpInputsProto"; | ||
| option java_multiple_files = true; | ||
| option (udpa.annotations.file_status).package_version_status = ACTIVE; | ||
|
|
||
| // [#protodoc-title: Common HTTP Inputs] | ||
|
|
||
| // Match input indicates that matching should be done on a specific request header. | ||
| // The resulting input string will be all headers for the given key joined by a comma, | ||
| // e.g. if the request contains two 'foo' headers with value 'bar' and 'baz', the input | ||
| // string will be 'bar,baz'. | ||
| // [#comment:TODO(snowp): Link to unified matching docs.] | ||
| message HttpRequestHeaderMatchInput { | ||
| // The request header to match on. | ||
| string header_name = 1; | ||
| } | ||
|
|
||
| // Match input indicating that matching should be done on a specific response header. | ||
| // The resulting input string will be all headers for the given key joined by a comma, | ||
| // e.g. if the response contains two 'foo' headers with value 'bar' and 'baz', the input | ||
| // string will be 'bar,baz'. | ||
| // [#comment:TODO(snowp): Link to unified matching docs.] | ||
| message HttpResponseHeaderMatchInput { | ||
| // The response header to match on. | ||
| string header_name = 1; | ||
| } |
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
35 changes: 35 additions & 0 deletions
35
generated_api_shadow/envoy/type/matcher/v3/http_inputs.proto
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
40 changes: 40 additions & 0 deletions
40
generated_api_shadow/envoy/type/matcher/v4alpha/http_inputs.proto
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,23 @@ | ||
| load( | ||
| "//bazel:envoy_build_system.bzl", | ||
| "envoy_cc_library", | ||
| "envoy_package", | ||
| ) | ||
|
|
||
| licenses(["notice"]) # Apache 2 | ||
|
|
||
| envoy_package() | ||
|
|
||
| envoy_cc_library( | ||
| name = "config", | ||
| srcs = ["config.cc"], | ||
| hdrs = ["config.h"], | ||
| deps = [ | ||
| "//include/envoy/registry", | ||
| "//include/envoy/server:filter_config_interface", | ||
| "//source/common/matcher:matcher_lib", | ||
| "//source/extensions/filters/http:well_known_names", | ||
| "//source/extensions/filters/http/common:factory_base_lib", | ||
| "@envoy_api//envoy/extensions/common/matching/v3:pkg_cc_proto", | ||
| ], | ||
| ) |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
?
Maybe just template this as you are likely going to end up with trailer versions also?