Re: svn commit: r1853407 - in /httpd/httpd/trunk/modules: http2/mod_proxy_http2.c proxy/mod_proxy_http.c

2019-02-14 Thread Stefan Eissing
Yann, The new v3 patch runs without any problems in my test setups. Nice work! As to the errors you see: that seems to be the output parsing for nghttp in the test classes that needs some more work. On your systems, DATA packages arrive different than my MacOS had seen so far. That messed up the

Re: svn commit: r1853407 - in /httpd/httpd/trunk/modules: http2/mod_proxy_http2.c proxy/mod_proxy_http.c

2019-02-13 Thread Yann Ylavic
Thanks Stefan, I think the "400" issue is fixed (r1853518, added to backport proposal), but two tests keep failing for in the test suite, namely test_004_post and test_500_proxy. They fail with or without my changes (trunk and 2.4.x), so I don't think it's related (mod_proxy does not seem to be re

Re: svn commit: r1853407 - in /httpd/httpd/trunk/modules: http2/mod_proxy_http2.c proxy/mod_proxy_http.c

2019-02-12 Thread Stefan Eissing
Ok, took the opportunity to add that to my pytest suite in mod-h2. You need a local "pytest", "curl" and "nghttp" executable. If you checkout https://github.com/icing/mod_h2 master, run the configure: > autoreconf -i > automake > autoconf > ./configure --with-apxs=/opt/apache-2.4.x/bin/apxs --en

Re: svn commit: r1853407 - in /httpd/httpd/trunk/modules: http2/mod_proxy_http2.c proxy/mod_proxy_http.c

2019-02-12 Thread Yann Ylavic
OK, thanks Stefan, will look at it. On Tue, Feb 12, 2019 at 11:12 AM ste...@eissing.org wrote: > > Yann, > > this works fine in my tests on trunk. However on 2.4.x I get often errors > when uploading data without content-length. > > Scenario from the h2 test suite, HTTP/2 on the front, old skoo

Re: svn commit: r1853407 - in /httpd/httpd/trunk/modules: http2/mod_proxy_http2.c proxy/mod_proxy_http.c

2019-02-12 Thread ste...@eissing.org
Yann, this works fine in my tests on trunk. However on 2.4.x I get often errors when uploading data without content-length. Scenario from the h2 test suite, HTTP/2 on the front, old skool HTTP/1.1 proxy to itself: > while true; do nghttp -v --expect-continue --data=gen/tmp/updata > -H'Conten