WIP: ext_authz: Use HTTP service URI#10573
WIP: ext_authz: Use HTTP service URI#10573dio wants to merge 5 commits intoenvoyproxy:masterfrom dio:fix-ext_authz-service-uri
Conversation
This patch enables the use of HTTP service URI. Signed-off-by: Dhi Aurrahman <dio@tetrate.io>
Signed-off-by: Dhi Aurrahman <dio@tetrate.io>
Signed-off-by: Dhi Aurrahman <dio@tetrate.io>
| // Settings used for controlling authorization response metadata. | ||
| AuthorizationResponse authorization_response = 8; | ||
|
|
||
| common.dynamic_forward_proxy.v2alpha.DnsCacheConfig dns_cache_config = 9; |
There was a problem hiding this comment.
Can you add some comment explaining how this is supposed to be used? Thanks.
|
This pull request has been automatically marked as stale because it has not had activity in the last 7 days. It will be closed in 7 days if no further activity occurs. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions! |
stale
Bot
added
the
stale
|
Please merge master to pick up #10672. We no longer accept changes to v2 (without explicit exception), so any API modifications should happen in v3. If this PR is adding a new proto, please follow the updated instructions in https://github.com/envoyproxy/envoy/blob/master/api/STYLE.md#adding-an-extension-configuration-to-the-api. |
stale
Bot
removed
the
stale
|
This pull request has been automatically marked as stale because it has not had activity in the last 7 days. It will be closed in 7 days if no further activity occurs. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions! |
stale
Bot
added
the
stale
|
This pull request has been automatically closed because it has not had activity in the last 14 days. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions! |
2 participants
Description:
This is a WIP to use the configured HTTP Service server URI as authorization service target.
Risk Level: Low
Testing: Unit, TBA integration
Docs Changes: TBA
Release Notes: TBA
Fixes #5357
Fixes #10104