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 t

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 ne

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 sec

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 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 dedicated code for &

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