bneradt commented on PR #10965: URL: https://github.com/apache/trafficserver/pull/10965#issuecomment-1879917048
> This pull request is intended to serve multiplexer plugin, as multiplexer plugin does not change the length of the payload. > > In order to support widely scene, it is better to enable / implement the re-chunk feature for Post Transform. I see. Thank you @oknet, this is a very helpful feedback and context. Reading through your explanation, I see two sets of things that can happen: 1. The multiplexer plugin should simply read the Content-Length header and use that for the `TSVConnWrite` api. This won't handled chunked POST requests of course, but it will at least fix things for large Content-Length bodies. And it correctly uses the `TSVConnWrite` api since it doesn't support INT64_MAX for POST requests. 2. If there are chunked requests, we'll have to handle that by updating the rechunk feature of Post Transform, as you state. For this pull request, I'll stick with simply addressing the first of those issues. This should cover the majority of contexts - in my experience, chunked POST requests are pretty rare in comparison to Content-Length. -- 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]
