[PATCH] Proxy: support configuration of socket buffer sizes

Maxim Dounin mdounin at mdounin.ru
Wed Dec 13 17:14:50 UTC 2017


On Tue, Dec 12, 2017 at 03:44:42PM +0000, Karstens, Nate wrote:

> I wanted to follow up on this because our software has been 
> released and so I can provide a few more details about our 
> setup.

Thank you for the details.


> I read through the Cloudflare blog entry (thank you -- that was 
> interesting). Their case is different because they were having 
> problems with a download, while we are having problems with an 
> upload. As such, perhaps the "proxy_send_timeout" setting 
> applies in this case instead of the "send_timeout" that they 
> were using. Still, it seems undesirable to increase the timeout 
> value to be too large because it increases the time before you 
> can detect a problem with the connection. The Cloudflare team 
> also considered reducing the TCP buffer size, but ultimately 
> decided on a different solution. I noticed that their solution 
> has not been incorporated into the main distribution, do you 
> know why that is?

The patch in question was never submitted for inclusion into 
nginx.  Likely because the authors consider it to be not ready for
inclusion.  I tend to agree.  While the approach looks 
interesting, it probably could be implemented in a more portable 
way and with less intrusion into generic code.

Maxim Dounin

More information about the nginx-devel mailing list