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

Rust Wasm Plugin Development Plan #1297

Open

007gzs opened this issue Sep 10, 2024 · 0 comments

Assignees

Labels

level/challenge sig/wasm

Collaborator

007gzs commented Sep 10, 2024 •

edited by johnlanni

Loading

proxy-wasm-rust-sdk

讨论 sdk 中不安全的unwrap()和panic是否需要修改为安全
- 上游的panic理论上都是不该出现的情况 Propagate the non-OK status instead of panic on hostcalls proxy-wasm/proxy-wasm-rust-sdk#83 Error in hostcalls functions causes filter to panic proxy-wasm/proxy-wasm-rust-sdk#82
- 部分错误应该在上游修改 [Bug] Wasm plugin crashed when calling get_http_request_headers() proxy-wasm/proxy-wasm-rust-sdk#217 Fixing map deserialization when header name or value is not a valid UTF8 string proxy-wasm/proxy-wasm-rust-sdk#188
plugin_wrapper中目前create_heep_context需要使用者主动效用create_heep_context_use_wrapper,讨论是否需要少量改动sdk来让使用者更方便

higress wasm-rust sdk

功能模块	难度	完成情况
`http_wrapper`	HttpCall 回调方案，rust生命周期管理导致用回调函数或创建struct当作回调object都不太好用	TODO
`redis_wrapper`	HttpCall 方案确认后可参考实现,sdk部分已增加未测试	TODO
`plugin_wrapper`	目前功能可用，细节考虑是否需要优化	TODO
`cluster_wrapper`	easy	TODO
`log_wrapper`	easy	TODO
`request_wrapper`	easy	TODO
`response_wrapper`	easy	TODO

The text was updated successfully, but these errors were encountered:

johnlanni changed the title ~~rust 插件讨论~~ Rust Wasm Plugin Development Plan

johnlanni assigned 007gzs

johnlanni added level/challenge sig/wasm labels

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment