Created attachment 34783 [details] Test for trailing space in status line RFC mandates for status line a mandatory space (U+0020) and optional reason phrase. Several tests do not check for or even trim the status line. The attached patch adds the space to the tests.
The tests in question aren't (explicitly) testing the format of the status line. They are testing that the correct status code is returned. I don't see any harm in expanding the tests to check for the space. I'll apply the patch shortly.
(In reply to Mark Thomas from comment #1) > The tests in question aren't (explicitly) testing the format of the status > line. They are testing that the correct status code is returned. > > I don't see any harm in expanding the tests to check for the space. I'll > apply the patch shortly. That's true, but some other tests do include the space character. This simply cleans up the remaining tests and adds the extra check for free. It is rather trivial.
Thanks. Patch applied.