wasm: Add data buffering for chunks #36411
Merged
CI (Envoy) / Mobile/CC
succeeded
Oct 14, 2024 in 2m 43s
Mobile/CC (success)
Check has finished
Details
Check run finished (success ✔️)
The check run can be viewed here:
Mobile/CC (pr/36411/main@dee70d1)
Check started by
Request (pr/36411/main@dee70d1)
dee70d1 
#36411 merge main@d5834c1
wasm: Add data buffering for chunks
Commit Message: wasm: Add data buffering for chunks
Additional Description:
In HTTP/2 connections, the last chunk, which contains the end_of_stream flag, is not used to call the Wasm callback. This fix addresses the issue by dumping the data into the buffer before calling the Wasm callback, ensuring that the data is now present.Risk Level: Low
Testing: yes
Docs Changes: n/a
Release Notes: yes
Platform Specific Features: n/aFixes #35884
Environment
Request variables
| Key | Value |
|---|---|
| ref | 5a20f4d |
| sha | dee70d1 |
| pr | 36411 |
| base-sha | d5834c1 |
| actor | @juanmolle |
| message | wasm: Add data buffering for chunks... |
| started | 1728928596.862089 |
| target-branch | main |
| trusted | false |
Build image
Container image/s (as used in this CI run)
| Key | Value |
|---|---|
| default | envoyproxy/envoy-build-ubuntu:f94a38f62220a2b017878b790b6ea98a0f6c5f9c |
| mobile | envoyproxy/envoy-build-ubuntu:mobile-f94a38f62220a2b017878b790b6ea98a0f6c5f9c |
Version
Envoy version (as used in this CI run)
| Key | Value |
|---|---|
| major | 1 |
| minor | 32 |
| patch | 0 |
| dev | true |
Loading