Skip to content

Dynamic Metadata for External Processing Filters Similar to Dynamic Metadata for External Authorization #19881

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

Closed
llu94 opened this issue Feb 8, 2022 · 7 comments · Fixed by #30747
Labels
area/ext_proc enhancement Feature requests. Not bugs or questions. stale stalebot believes this issue/PR has not been touched recently

Comments

@llu94
Copy link
Contributor

llu94 commented Feb 8, 2022

Title: Dynamic Metadata for External Processing Filters

Description:
Dynamic Metadata emissions for ext_proc like dynamic metadata in ext_authorization

[optional Relevant Links:]
>Any extra documentation required to understand the issue.

@llu94 llu94 added enhancement Feature requests. Not bugs or questions. triage Issue requires triage labels Feb 8, 2022
@llu94 llu94 changed the title Dynamic Metadata for External Processing Filters Similar External Authorization Dynamic Metadata for External Processing Filters Similar to Dynamic Metadata for External Authorization Feb 8, 2022
@htuch htuch added area/ext_proc and removed triage Issue requires triage labels Feb 9, 2022
@llu94
Copy link
Contributor Author

llu94 commented Feb 9, 2022

Basically what this says on the tin. I'd be happy to work on this but I'd like to ask the code owners what they think about this. I am planning on basically just porting the behavior from ext_authz filters to ext_proc filter.

@pgeler
Copy link

pgeler commented Feb 19, 2022

+1 on this, having a similar use case, using "header to metadata" filter instead.

@gbrail any thoughts about this?

@github-actions
Copy link

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 "no stalebot" or other activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale stalebot believes this issue/PR has not been touched recently label Mar 22, 2022
@llu94
Copy link
Contributor Author

llu94 commented Mar 22, 2022

I'm ready to start work on this. @pgeler can you elaborate more about your use case?

@github-actions github-actions bot removed the stale stalebot believes this issue/PR has not been touched recently label Mar 22, 2022
@github-actions
Copy link

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 "no stalebot" or other activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale stalebot believes this issue/PR has not been touched recently label Apr 21, 2022
@github-actions
Copy link

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" or "no stalebot". Thank you for your contributions.

@cetanu
Copy link
Contributor

cetanu commented Oct 13, 2022

Sorry to resurrect but I believe this would still be valuable

If additional specifications are needed, I would describe them as follows:

Background:
The External Authorization service receives a CheckRequest.
The CheckRequest contains an AttributeContext, which has a field metadata_context of type Metadata

Desire/suggestion:
It would be quite useful if a similar field could be included in the ProcessingRequest sent to an External Processor service

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ext_proc enhancement Feature requests. Not bugs or questions. stale stalebot believes this issue/PR has not been touched recently
Projects
None yet
4 participants