Probably want to check the handling of the Connection header. Don't need to use "Connection: close" for a 304 for example?
Might as well handle the other status that also don't want a content length: 100, 101, 102, 204?
Should the approach be more forceful and explicitly remove any Content-Length header? (At the moment the patch avoids adding Content-Length but that does not stop application code from getting it wrong.)
Some thoughts about the patch:
Probably want to check the handling of the Connection header. Don't need to use "Connection: close" for a 304 for example?
Might as well handle the other status that also don't want a content length: 100, 101, 102, 204?
Should the approach be more forceful and explicitly remove any Content-Length header? (At the moment the patch avoids adding Content-Length but that does not stop application code from getting it wrong.)