On Sep 10, 2013, at 6:54 AM, Igor Galić <[email protected]> wrote:

> 
>> +    /* Short circuit the / path, common case, and we won't allow healthecks
>> on / */
>> +    if (!path || !path_len)
> 
> shouldn't this error out before going to cleanup?

No, it's not an error case, just an "optimization" such that on requests for 
"/", we don't even try to match up against possible health check URLs. The 
original bug reported used this as a bug fix, but that only hides the real bug 
(which is fixed in a separate commit).

-- leif

Reply via email to