On 08/23/2016 08:08 AM, Amos Jeffries wrote: > A followup patch can be done to give skipDelimiter a 'const char* which' > parameter that takes a description/name for the delimiter to improve > that debug output. > > so: > skipDelimiter(blah, "method") > > produces: > invalid request-line: missing method delimiter
Good idea! I would s/which/where/ s/missing <which> delimiter/missing delimiter <where>/ because the URI field has two associated delimiters (left and right), and a "where" parameter would be able to detail the missing delimiter better: invalid request-line: missing delimiter after method invalid request-line: missing delimiter before "HTTP/1" Same for the "too many delimiters" error, of course. Thank you, Alex. _______________________________________________ squid-dev mailing list squid-dev@lists.squid-cache.org http://lists.squid-cache.org/listinfo/squid-dev