Skip to content

Comments

add KDP for HTTP authorization#78

Open
aerosouund wants to merge 3 commits intokyverno:mainfrom
aerosouund:http-auth
Open

add KDP for HTTP authorization#78
aerosouund wants to merge 3 commits intokyverno:mainfrom
aerosouund:http-auth

Conversation

@aerosouund
Copy link
Member

No description provided.

Signed-off-by: aerosouund <aerosound161@gmail.com>
evaluation:
mode: HTTP
validations:
- expression: >
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As we reusing the same API (definition) an expression has to return a boolean value, right?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you're referring to the fact that CEL expressions need to return a bool then no. It has to return a http.response type or null. To avoid the conflict, the envoy plugin (or whatever will handle HTTP authorization) will not handle policies that don't have evaluation.mode: HTTP. Likewise, kyverno won't handle policies that have this enabled

Copy link
Member

@fjogeleit fjogeleit Sep 2, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

and in which sense is it a validation when you returning/building a response object rather then do a validation? This also means that you can not reuse the existing VPOL engine?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah exactly, there's another compiler implementation in the envoy plugin built for those use cases and thats what i am targeting. The plugin also recently added support for VPOL. It is validation in the sense that i am returning a response to a after (validating) a request

Signed-off-by: aerosouund <aerosound161@gmail.com>
Signed-off-by: aerosouund <aerosound161@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants