Hi, I wasn't sure of any other examples of it breaking the HTTP standard, so figured a compile time option was safer. Unfortunately bugging Google any more than we already do probably won't get the problem solved, and the standard "100 Continue" behavior causes 502 errors for our end users.
For reference here is the issue on the google side: https://code.google.com/p/google-compute-engine/issues/detail?id=298 If you still deem it useful I can update this to use an "expect_continue"? option instead. - josh On Wed, Aug 31, 2016 at 10:38 AM, Maxim Dounin <[email protected]> wrote: > Hello! > > On Wed, Aug 31, 2016 at 10:08:44AM -0700, Josh Yudaken wrote: > >> # HG changeset patch >> # User Josh Yudaken <[email protected]> >> # 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/ > > _______________________________________________ > nginx-devel mailing list > [email protected] > http://mailman.nginx.org/mailman/listinfo/nginx-devel _______________________________________________ nginx-devel mailing list [email protected] http://mailman.nginx.org/mailman/listinfo/nginx-devel
