Changed how HTTP headers are ended on Linux

Open maxwalley opened this issue 2 years ago • 2 comments

This is fixing the error reported at https://forum.juce.com/t/url-header-suffix-on-linux/59985/2

Essentially the HTTP header section should end with "/r/n/r/n". The previous code was not taking into account that there might already be some of the suffix present before applying it.

Feb 15 '24 10:02 maxwalley

so what's the progress on this PR? We face the same issue on Linux that JUCE sends request with extra /r/n and it breaks parsing of request on our server

Jun 16 '25 10:06 sevoster

We currently have had to move to a forked version of JUCE found here: https://github.com/salsa-sound/JUCE-6

This request has now been open for over a year with no resolution.

Jun 16 '25 10:06 maxwalley

Thanks for the issue report, there's now a fix available: https://github.com/juce-framework/JUCE/commit/b8e0146a3c3de594f150356f42cfdb95064d62d3

Jun 19 '25 15:06 reuk