Re: Compliance: reply with 400 (Bad Request) if request header is too big

2010-08-30 Thread Henrik Nordström
The added comment applies to the whole 6xx class, not just 601. Note: Not entirely sure we need these as internal http status codes in squid-3, but that's separate from this change. sön 2010-08-29 klockan 14:57 -0600 skrev Alex Rousskov: HTTP_INSUFFICIENT_STORAGE = 507,/** RFC2518

Re: Compliance: reply with 400 (Bad Request) if request header is too big

2010-08-30 Thread Alex Rousskov
On 08/30/2010 01:44 AM, Henrik Nordström wrote: The added comment applies to the whole 6xx class, not just 601. We have only two 6xx constants. Are you saying HTTP_INVALID_HEADER (600) is never sent to the client? I will adjust the comment if that is the case. Note: Not entirely sure we

Re: Compliance: reply with 400 (Bad Request) if request header is too big

2010-08-30 Thread Henrik Nordström
- Ursprungsmeddelande - On 08/30/2010 01:44 AM, Henrik Nordström wrote: The added comment applies to the whole 6xx class, not just 601. We have only two 6xx constants. Are you saying HTTP_INVALID_HEADER (600) is never sent to the client? I will adjust the comment if that is the

Compliance: reply with 400 (Bad Request) if request header is too big

2010-08-29 Thread Alex Rousskov
Compliance: reply with 400 (Bad Request) if request header is too big. Reply with a standard 400 (Bad Request) instead of 601 (Unknown) status in case of an ERR_TOO_BIG error. HTTP does not have a dedicated code for the too-big header error. There is 414 (Request-URI Too Long), but Squid does

Re: Compliance: reply with 400 (Bad Request) if request header is too big

2010-08-29 Thread Amos Jeffries
On Sun, 29 Aug 2010 14:57:08 -0600, Alex Rousskov rouss...@measurement-factory.com wrote: Compliance: reply with 400 (Bad Request) if request header is too big. Reply with a standard 400 (Bad Request) instead of 601 (Unknown) status in case of an ERR_TOO_BIG error. HTTP does not have a