yangjj-iso commented on PR #7569: URL: https://github.com/apache/shenyu/pull/7569#issuecomment-6136166847
@wy471x I addressed the coordination and compatibility points in 1fc74a273, synced with master at a89459bf2. This PR now uses the strict lookup policy already merged in #7568: missing exports return empty, while a same-named non-function export throws InvalidException. There is no WasmLoader production change left in this diff, and the lookup test agrees with #7568's regression. #7567 retains the shared private helper only for consistent initializer validation; this bridge PR no longer duplicates a competing lookup implementation. The README and release note explicitly describe the no-argument constructor's new JSON semantics, strict parsing/1 MiB/HTTP 500 behavior, the explicit-null constructor path for retaining no-op callbacks, custom-adapter compatibility, concrete continuation/ID defaults, and opt-in destruction wiring. I added comments explaining the loader lock and response-header behavior, and strengthened the eager-adapter regression to assert that the guest's put_result callback was entered before subscribing. The new fixture CI workflow pins WABT 1.0.42 and Rust 1.90.0. Local verification reproduced all three WAT binaries exactly, rebuilt the five Rust guests and passed 89 WASM tests against them, with zero failures/errors/skips. An intentionally corrupted WAT binary is rejected by the verifier. Checkstyle, Apache RAT and the pinned license checker pass. The two sibling branches have no current conflicts with master; new CI is running. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
