Consider using "proxy_set_header Connection $http_connection;" in all docs and examples
|Reported by:||Owned by:|
|uname -a:||(none, this is a website change)|
|nginx -V:||(none, this is a website change)|
There are a number of places in shipped config and documentation that include the following config line:
proxy_set_header Connection "upgrade";
Please consider changing it instead to:
proxy_set_header Connection $http_connection;
Connection: upgrade seems to cause failures in some strict-parsing servers, particularly anything based on ASP.NET which rejects the request at a framework level.
Please see this github issue for some more discussion, and reports from affected people.
I know the fault really lies with administrators and container-authors blindly copying and pasting config from examples, but I think it would indirectly help a lot of people if you changed the examples.