Welcome! Log In Create A New Profile

Advanced

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

Maxim Dounin
August 31, 2016 01:40PM
Hello!

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

> # HG changeset patch
> # User Josh Yudaken <josh@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/

_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel
Subject Author Views Posted

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

Josh Yudaken 520 August 31, 2016 01:16PM

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

Maxim Dounin 172 August 31, 2016 01:40PM

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

Josh Yudaken 205 August 31, 2016 02:02PM



Sorry, you do not have permission to post/reply in this forum.

Online Users

Guests: 95
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready