nic-6443 opened a new pull request, #13428:
URL: https://github.com/apache/apisix/pull/13428

   ### Description
   
   `core.request.get_body()` still rejects HTTP/2 and HTTP/3 requests without a 
`Content-Length` header, but that guard was added for an older lua-nginx-module 
behavior that has since been reverted upstream.
   
   Current lua-nginx-module no longer rejects `ngx.req.read_body()` for HTTP/2 
or HTTP/3 requests without `Content-Length`, so APISIX should not reject those 
requests at its own request helper layer. This also matches the discussion in 
#11362, where the reported behavior was already stale for newer OpenResty 
releases.
   
   This removes the stale guard and drops the test-only `Content-Length: 0` 
headers that were added for it.
   
   #### Which issue(s) this PR fixes:
   
   Fixes #11362
   
   ### Checklist
   
   - [x] I have explained the need for this PR and the problem it solves
   - [x] I have explained the changes or the new features added to this PR
   - [x] I have added tests corresponding to this change
   - [x] I have updated the documentation to reflect this change (not needed; 
this removes an internal stale guard and there is no documented config/API 
change)
   - [x] I have verified that this change is backward compatible (If not, 
please discuss on the [APISIX mailing 
list](https://github.com/apache/apisix/tree/master#community) first)
   


-- 
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]

Reply via email to