bneradt commented on PR #10965:
URL: https://github.com/apache/trafficserver/pull/10965#issuecomment-1881440075

   > By review your current code, let me confirm the implement and details with 
you:
   > 
   > 1. dispatch HTTP request directly except ( POST and PUT )
   > 2. For the POST and PUT request with valid `Content-Length` header,
   >    a. Collect data into `s.origin_buffer` by transform plugin,
   >    b. Get all data by `s.clone_reader` while the transformVC closed,
   >    c. Then dispatch the request.
   > 3. For the POST and PUT request without `Content-Length` header, does not 
dispatch the request.
   > 
   > I'll approve your commit if the above list is all you have done at this 
stage.
   
   Yes, this is correct. All that my current patch does now is set the length 
explicitly in the `TSVConnWrite` because that's what the API currently requires 
for POST bodies. Basically, this current patch fixes multiplexer to behave like 
the current API requires. Which is at least a step in a good direction, because 
it make multiplexer work at least for Content-Length bodies.
   


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