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

Add more cel extensions #1717

Closed
venc0r opened this issue Apr 15, 2024 · 5 comments · Fixed by #1730
Closed

Add more cel extensions #1717

venc0r opened this issue Apr 15, 2024 · 5 comments · Fixed by #1730
Assignees
Labels
kind/feature Categorizes issue or PR as related to a new feature.

Comments

@venc0r
Copy link

venc0r commented Apr 15, 2024

Feature request

the docs say the cel interceptor should support all extensions

All the functionality from the cel-go project’s CEL extension is available in your CEL expressions.

Over the last months there are new extensions added to cel, but not enabled in the cel-eval interceptor.

Use case

I want to get a slice of an array like: [1,2,3,4].slice(1,3) => [2,3]

@venc0r venc0r added the kind/feature Categorizes issue or PR as related to a new feature. label Apr 15, 2024
@khrm
Copy link
Contributor

khrm commented Apr 16, 2024

@venc0r Won't a update to the latest library fix this? #1718

@khrm khrm linked a pull request Apr 16, 2024 that will close this issue
7 tasks
@khrm khrm mentioned this issue Apr 16, 2024
7 tasks
@khrm khrm removed a link to a pull request May 7, 2024
7 tasks
@khrm
Copy link
Contributor

khrm commented May 7, 2024

/assign @khrm

@khrm
Copy link
Contributor

khrm commented May 7, 2024

/assign @bigkevmcd

@khrm khrm removed their assignment May 7, 2024
@khrm
Copy link
Contributor

khrm commented May 8, 2024

It will be fixed by #1730

@khrm khrm linked a pull request May 8, 2024 that will close this issue
7 tasks
@bigkevmcd
Copy link
Member

@venc0r #1730 will add most of the new CEL extensions, I deliberately didn't add the Proto extension as I'm unsure of a use case for it, but the slices, sets and math extensions are now in there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue or PR as related to a new feature.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants