id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc	uname	nginx_version
2115	"Consider using ""proxy_set_header Connection $http_connection;"" in all docs and examples"	sbrudenell@…		"Hi,

There are a number of places in shipped config and documentation that include the following config line:

{{{
proxy_set_header Connection ""upgrade"";
}}}

(especially: [https://nginx.org/en/docs/http/websocket.html here])

Please consider changing it instead to:

{{{
proxy_set_header Connection $http_connection;
}}}

An unconditional {{{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 [https://github.com/dotnet/aspnetcore/issues/17081 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.

Thanks!"	enhancement	closed	minor		documentation	1.19.x	wontfix		sbrudenell@…	(none, this is a website change)	(none, this is a website change)