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.

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

ext_authz: allow receiving dynamic metadata from gRPC authorization server #9049

Closed
erikbos opened this issue Nov 16, 2019 · 4 comments · Fixed by #11820
Closed

ext_authz: allow receiving dynamic metadata from gRPC authorization server #9049

erikbos opened this issue Nov 16, 2019 · 4 comments · Fixed by #11820
Labels
area/ext_authz design proposal Needs design doc/proposal before implementation help wanted Needs help!

Comments

@erikbos
Copy link
Contributor

erikbos commented Nov 16, 2019

Title: ext_authz: enable receiving dynamic metadata from gRPC authorization server

Description:
I've got a custom authorization server which looks up some additional data for each request that I like to carry forward to other filters, but not upstream.

Example use cases for setting metadata:

  1. Setting dynamic rate limit value, authorization server has bespoke logic to determine rate limiting setting for request (leverages Proposal: support for dynamic rate limits #11595)
  2. Passing on internal authorization server fields which should end up in the access log (e.g. customerId, "rate limiter plan")

Relevant links
I think this might be the opposite of feature request #7699 (Fixed with PR #7818)

@mattklein123 mattklein123 added the design proposal Needs design doc/proposal before implementation label Nov 17, 2019
@mattklein123
Copy link
Member

cc @dio @gsagula

@stale
Copy link

stale bot commented Dec 17, 2019

This issue has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in the next 7 days unless it is tagged "help wanted" or other activity occurs. Thank you for your contributions.

@stale stale bot added the stale stalebot believes this issue/PR has not been touched recently label Dec 17, 2019
@stale
Copy link

stale bot commented Dec 24, 2019

This issue has been automatically closed because it has not had activity in the last 37 days. If this issue is still valid, please ping a maintainer and ask them to label it as "help wanted". Thank you for your contributions.

@stale stale bot closed this as completed Dec 24, 2019
@dio dio added the help wanted Needs help! label Dec 24, 2019
@dio dio reopened this Dec 24, 2019
@stale stale bot removed the stale stalebot believes this issue/PR has not been touched recently label Dec 24, 2019
@erikbos
Copy link
Contributor Author

erikbos commented Jun 30, 2020

@Pchelolo FYI

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ext_authz design proposal Needs design doc/proposal before implementation help wanted Needs help!
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants