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.

Sign up for GitHub

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

Jump to bottom

Add some (type safe?) way to get to host's properties in CEL #119

Closed
alexsnaps opened this issue Oct 25, 2024 · 0 comments · Fixed by #130
Closed

Add some (type safe?) way to get to host's properties in CEL #119

alexsnaps opened this issue Oct 25, 2024 · 0 comments · Fixed by #130
Assignees
@alexsnaps

Comments

@alexsnaps
Copy link
Member

Have a getHostProperty([string]): bytes at the very least... but possibly have some type conversion covered by the helper?
Cause bytes can only be converted to string as of now, if I read the spec correctly...
@alexsnaps alexsnaps self-assigned this Nov 2, 2024
alexsnaps added a commit that referenced this issue Nov 2, 2024
@alexsnaps
Fix #119
453dbcc 
Signed-off-by: Alex Snaps <[email protected]>
@alexsnaps alexsnaps linked a pull request Nov 2, 2024 that will close this issue
Fix #119 #130
Merged
alexsnaps added a commit that referenced this issue Nov 4, 2024
@alexsnaps
Fix #119
857ff10 
Signed-off-by: Alex Snaps <[email protected]>
alexsnaps added a commit that referenced this issue Nov 5, 2024
@alexsnaps
Merge pull request #130 from Kuadrant/get_prop
dd689d7 
Fix #119
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alexsnaps alexsnaps

Labels
None yet
Projects
Kuadrant
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #119 Kuadrant/wasm-shim
1 participant
@alexsnaps