On Tue, Jul 20, 2021 at 5:28 PM Yann Ylavic <[email protected]> wrote:
>
> On Tue, Jul 20, 2021 at 5:19 PM Ruediger Pluem <[email protected]> wrote:
> >
> > Yes, but as pointed out in my initial reply I think the crashes are not
> > related to your code changes but the mentioned missing
> > backports.
>
> We could probably limit changes in proxy_balancer.t (r1891321) to 2.5
> until then..
Something like the attached?
Index: t/modules/proxy_balancer.t
===================================================================
--- t/modules/proxy_balancer.t (revision 1891689)
+++ t/modules/proxy_balancer.t (working copy)
@@ -107,7 +107,8 @@ if (have_min_apache_version("2.4.41")) {
}
# Try with the referer and http (byrequests)
-if (have_module('lbmethod_byrequests')) {
+# XXX: crashes in 2.4.48, when r1891477 is backported change "2.5.1" requirement to the appropriate "2.4.x" version.
+if (have_min_apache_version("2.5.1") && have_module('lbmethod_byrequests')) {
$r = GET("/dynproxy");
ok t_cmp($r->code, 503, "request should fail for /dynproxy");
# create it