nic-6443 commented on issue #13237: URL: https://github.com/apache/apisix/issues/13237#issuecomment-4676896520
This is fixed on master: the H2/H3 Content-Length guard in `core.request.get_body()` was removed entirely in #13428 (commit 8048e7447), so ext-plugin-post-resp — and every other plugin calling `get_body()` — no longer returns 502 for HTTP/2 requests without a body. The reasoning matches what you found in the lua-nginx-module history: the `ngx.req.read_body()` restriction the guard was protecting against has been lifted in the ngx_lua versions APISIX now builds against (master is on apisix-runtime 1.3.6 / OpenResty 1.29.2.4), so the check was stale. The fix merged after 3.16.0 was cut, so it isn't in any released version yet — it'll be in the next release. Since the guard is gone from core, the workaround in #13243 is no longer needed and both could be closed. Thanks for tracing this all the way down to the lua-nginx-module commits. -- 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]
