[PATCH] Add --without-http-expect option

Maxim Dounin mdounin at mdounin.ru
Wed Aug 31 17:38:46 UTC 2016


Hello!

On Wed, Aug 31, 2016 at 10:08:44AM -0700, Josh Yudaken wrote:

> # HG changeset patch
> # User Josh Yudaken <josh at smyte.com>
> # Date 1472606412 25200
> #      Tue Aug 30 18:20:12 2016 -0700
> # Node ID f81a22addc7fe1ca7486e932470756f4d73a3bb2
> # Parent  c6372a40c2a731d8816160bf8f55a7a50050c2ac
> Add --without-http-expect option
> 
> Some load balancers (specifically Google Cloud) do not support the expect
> header. This adds an option to compile nginx without support.

It is not clear why you are trying to add this as compile-time 
option.  The Expect header is defined by the HTTP standard, and 
compiling nginx without appropriate support looks clearly wrong.

If Google Cloud fails to properly balance HTTP and needs special 
workarounds on nginx side, you may consider adding this as a 
run-time option instead, similar to the 
"chunked_transfer_encoding" directive, see 
http://nginx.org/r/chunked_transfer_encoding.

Or, better yet, report the bug to Google.  It should be more or 
less trivial for them to fix this by just removing the Expect 
header from forwarded requests.

-- 
Maxim Dounin
http://nginx.org/



More information about the nginx-devel mailing list