]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MAJOR: http: fix regression introduced by commit d655ffe
authorWilly Tarreau <w@1wt.eu>
Tue, 2 Apr 2013 22:22:25 +0000 (00:22 +0200)
committerWilly Tarreau <w@1wt.eu>
Tue, 2 Apr 2013 22:22:25 +0000 (00:22 +0200)
commit2d43e18b6927b73aa9b1e6c8a40fc5b235498a5f
tree42252a6b6c4044f993ab1f83b9c8f5137d10a341
parentffb6f08babe3252c73995d1c7f2fba096fa7b370
BUG/MAJOR: http: fix regression introduced by commit d655ffe

Sander Klein reported that since last snapshot, some downloads would
hang from nginx but succeed from apache. The culprit was not too hard
to find given the low number of recent changes affecting the data path.

Commit d655ffe slightly reorganized the HTTP state machine and
introduced this regression. The reason is that we must never jump
into the MSG_DONE case without first flushing remaining data because
this is not done anymore afterwards. This part is scheduled for
being reorganized since it's totally ugly especially since we added
compression, and this regression is an illustration of its readability.

The issue is entirely dependant on the server close sequence, which
explains why it was reproducible only with nginx here.
src/proto_http.c