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

Support for Common Expression Language (CEL) #475

Open
guicassolato opened this issue Jul 9, 2024 · 1 comment
Open

Support for Common Expression Language (CEL) #475

guicassolato opened this issue Jul 9, 2024 · 1 comment
Labels
Milestone

Comments

@guicassolato
Copy link
Collaborator

Add support for Common Expression Language (CEL) as alternative for all selectors currently based on tidwall/gjson, including:

CEL as a superset of gjson-powered Authorino features

TBC – All gjson-powered features supported in Authorino can be modeled and supported in CEL. Special attention to be given to gjson's modifiers and custom ones introduced by Authorino.

Should CEL paths become the standard?

To keep backward compatibility, maybe the paths specified in CEL can be prefixed with cel:, until slowly transitioned to become the default, when either we can drop support to gjson or introduce a prefix for it to keep being supported.

@alexsnaps
Copy link
Member

wrt modifiers, I think these can be custom functions we add to the cel interpreter. I'd suggest we keep these align across the "runtimes" tho, i.e. have them being kuadrant-wide extensions (wasm-shim, limitador, ...)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Todo
Development

No branches or pull requests

2 participants