-
Notifications
You must be signed in to change notification settings - Fork 24
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
Regex support? #48
Comments
Definitely not intentional, it's been more a lack of demand than anything. I'll definitely get this on the priority list, or I would be happy to work through a PR with you if you'd like to take a stab at it as well. Otherwise I'll take a look later this week. |
Great! I'm certainly willing to have a shot at this, as if I can make it work here, I don't have to support two different code path in my own on-going work (see here, tho I'm rewriting the entire support for typing leveraging CEL, as that's how many types are actually represented with Envoy anyways). |
Looks like regex would do the job... I'll start looking into leveraging it to add support for regex in the interpreter |
This is more of a question really than an actual issue... for now at least.
If I read the cel spec properly, it is expected it'd support regular expressions, re2 based flavor.
Trying this out:
I'm getting a
Err(NoSuchKey("matches"))
for bothmap
&string
tests. Couldn't see anything mentioning regular expressions neither. So here's the questions, is this a conscious decision to not support them? Looking around, I couldn't find a good candidate regex lib to start implementing that support actually... especially as in our use-case we're looking at targeting wasm...Have you considered that side of the spec? Any conclusion you came to already?
The text was updated successfully, but these errors were encountered: