On 05/15/2013 08:47 PM, Amos Jeffries wrote: > On 16/05/2013 5:14 a.m., Alex Rousskov wrote: >> On 05/15/2013 07:41 AM, Amos Jeffries wrote: >>> On 16/05/2013 1:06 a.m., Martin Sperl wrote: >>>> Hi! >>>> >>>> We came across a strange "bug" in squid in regards to icap and >>>> request_header access. >>> It is a bug alright. Header names are supposed to be case-insensitive. >> The attached patches fix this bug in Squid v3.2 and trunk. >> >> Disclaimers: I only tested the trunk patch. I do not think this is >> related to ICAP (the bug affects all Squid configurations that use >> header_access directives). >> >> >> HTH, >> >> Alex. >> > > "NoCaseLess" seems a very weird thing to call the insensitive comparator > - you may want to rethink that. But +1 anyway. It is a "less" operator that disregards case. IIRC similar names are quite common in this context (because "CaseInsensitive" is too long and "iCase" may remind folks of a certain product line). I added "Than" to clarify: NoCaseLessThan but feel free to rename. Committed to trunk as r12828. Thank you, Alex.
