On Aug 12, 2008, at 5:21 PM, mavit wrote:

>
> andyg;328802 Wrote:
>> An HTTP header ends with 2 CRLF's. We were only adding 1 before this
>> change. This is moot on SC because the whole request gets filtered
>> through HTTP::Request which will fix any problem like that anyway.
>> The problem only became apparent on SN where HTTP::Request isn't used
>>
>> here.

>
> Ah, but I count three CRLFs. This effectively gives you a normal
> header followed by a body consisting of a single CRLF. GET requests
> aren't supposed to have bodies, hence the 400 Bad Request response.
>
>> From Wireshark:

>
> Code:
> --------------------
>
> 0000 47 45 54 20 2f 20 48 54 54 50 2f 31 2e 30 0d 0a GET / HTTP/
> 1.0..
> 0010 43 61 63 68 65 2d 43 6f 6e 74 72 6f 6c 3a 20 6e Cache-
> Control: n
> 0020 6f 2d 63 61 63 68 65 0d 0a 43 6f 6e 6e 65 63 74 o-
> cache..Connect
> 0030 69 6f 6e 3a 20 63 6c 6f 73 65 0d 0a 41 63 63 65 ion:
> close..Acce
> 0040 70 74 3a 20 2a 2f 2a 0d 0a 48 6f 73 74 3a 20 68 pt: */
> *..Host: h
> 0050 61 6e 64 6d 61 67 69 63 2e 6d 75 78 74 61 70 65
> andmagic.muxtape
> 0060 2e 63 6f 6d 0d 0a 55 73 65 72 2d 41 67 65 6e 74 .com..User-
> Agent
> 0070 3a 20 69 54 75 6e 65 73 2f 34 2e 37 2e 31 20 28 : iTunes/
> 4.7.1 (
> 0080 4c 69 6e 75 78 3b 20 4e 3b 20 52 65 64 20 48 61 Linux; N;
> Red Ha
> 0090 74 3b 20 69 36 38 36 2d 6c 69 6e 75 78 3b 20 45 t; i686-
> linux; E
> 00a0 4e 3b 20 75 74 66 38 29 20 53 71 75 65 65 7a 65 N; utf8)
> Squeeze
> 00b0 43 65 6e 74 65 72 2f 37 2e 31 2f 32 32 31 37 30 Center/
> 7.1/22170
> 00c0 0d 0a 49 63 79 2d 4d 65 74 61 64 61 74 61 3a 20 ..Icy-
> Metadata:
> 00d0 31 0d 0a 0d 0a 0d 0a 1......


You're right, I stand corrected. Fixed in 7.2 change 22563.