Hi Tim,

thanks for the review.

On Sun, Nov 15, 2020 at 1:50 PM Tim Düsterhus <t...@bastelstu.be> wrote:
> Am 15.11.20 um 13:45 schrieb William Dauchy:
> > +#REQUIRE_OPTION=OPENSSL
>
> I don't believe OPENSSL is required. The test passes for me even without
> compiling with SSL.

true bad copy/paste on my side.

> > +client c1 -connect ${h1_fe_sock} {
> > +     txreq -url "/bla+?foo%3Dbar%2B42+42"
>
> Maybe add %20 to the string to test the other variant of encoding a space?

fixed in v2 even though %20 is not much different than other encoded
characters to my knowledge.
--
William

Reply via email to