cloud-custodian / cloud-custodian/cel-python

'MessageType' support

Open
#43 2 comments 0 reactions 1 assignee Claimed by @Alfus View on GitHub
enhancement help wanted
Dominant language
Python
Stars
174
Forks
40
PR merge metrics
No merged PRs in 30d

Description

I am in the process of implementing protovalidate in pure-python using cel-python have push things as far as I can from the outside leaving me the following 7 issues:

![image](https://github.com/cloud-custodian/cel-python/assets/4567082/67a8871a-ba75-444d-8881-28113c424ce4)

1-2, 5-7: cel-python doesn't support nano resolution
3: cel-python doesn't support protobuf messages
4: cel-python doesn't check the data is valid before applying the regex

I am mostly interested (3). I have implemented most of the logic needed to seamlessly integrate protobuf into cel-python, by materializing protobuf messages as a map; however, as protobuf supports cycles in the type system, this is insufficient (as well as inefficient). I am curious if there is any interest to add support, potentially based on the code I have already written.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.