Re: Broken: apache/httpd#1754 (2.4.x - 0244d12)

2021-07-20 Thread Yann Ylavic
On Tue, Jul 20, 2021 at 5:51 PM Ruediger Pluem  wrote:
>
> Looks sensible.

r1891690.

Regards;
Yann.


Re: Broken: apache/httpd#1754 (2.4.x - 0244d12)

2021-07-20 Thread Ruediger Pluem
Looks sensible.

Regards

Rüdiger

On 7/20/21 5:45 PM, Yann Ylavic wrote:
> On Tue, Jul 20, 2021 at 5:28 PM Yann Ylavic  wrote:
>>
>> On Tue, Jul 20, 2021 at 5:19 PM Ruediger Pluem  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?
> 


Re: Broken: apache/httpd#1754 (2.4.x - 0244d12)

2021-07-20 Thread Yann Ylavic
On Tue, Jul 20, 2021 at 5:28 PM Yann Ylavic  wrote:
>
> On Tue, Jul 20, 2021 at 5:19 PM Ruediger Pluem  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


Re: Broken: apache/httpd#1754 (2.4.x - 0244d12)

2021-07-20 Thread Yann Ylavic
On Tue, Jul 20, 2021 at 5:19 PM Ruediger Pluem  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..

Cheers;
Yann.


Re: Broken: apache/httpd#1754 (2.4.x - 0244d12)

2021-07-20 Thread Ruediger Pluem
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.

Regards

Rüdiger

On 7/20/21 5:11 PM, Stefan Eissing wrote:
> Did I just trigger the built with my changes? Because I had no changes 
> related to mod_proxy...
> 
>> Am 20.07.2021 um 16:56 schrieb Travis CI :
>>
>> apache / httpd
>> 2.4.x
>> Build #1754 was broken25 mins and 20 secs
>> Stefan Eissing0244d12 CHANGESET →
>> Merged /httpd/httpd/trunk:r1891683
>>
>> *) mod_md:
>> - Domain names in `` can now appear in quoted form.
>> - Fixed a failure in ACME challenge selection that aborted further searches
>> when the tls-alpn-01 method did not seem to be suitable.
>> - Changed the tls-alpn-01 setup to only become unsuitable when none of the
>> dns names showed support for a configured 'Protocols ... acme-tls/1'. This
>> allows use of tls-alpn-01 for dns names that are not mapped to a VirtualHost.
>>
>>
>>
>> git-svn-id: 
>> https://svn.apache.org/repos/asf/httpd/httpd/branches/2.4.x@189168513f79535-47bb-0310-9956-ffa450edef68
>> Want to know about upcoming build environment updates?
>>
>> Would you like to stay up-to-date with the upcoming Travis CI build 
>> environment updates? We set up a mailing list for you!
>>
>> SIGN UP HERE
>>  Documentation about Travis CI
>> Have any questions? We're here to help.
>> Unsubscribe from build emails from the apache/httpd repository.
>> To unsubscribe from all build emails, please update your settings.
>>
>> Travis CI GmbH, Rigaer Str. 8, 10427 Berlin, Germany | GF/CEO: Randy Jacops 
>> | Contact: cont...@travis-ci.com | Amtsgericht Charlottenburg, Berlin, HRB 
>> 140133 B | Umsatzsteuer-ID gemäß §27 a Umsatzsteuergesetz: DE282002648
> 
> 


Re: Broken: apache/httpd#1754 (2.4.x - 0244d12)

2021-07-20 Thread Stefan Eissing
Did I just trigger the built with my changes? Because I had no changes related 
to mod_proxy...

> Am 20.07.2021 um 16:56 schrieb Travis CI :
> 
> apache / httpd
> 2.4.x
> Build #1754 was broken25 mins and 20 secs
> Stefan Eissing0244d12 CHANGESET →
> Merged /httpd/httpd/trunk:r1891683
> 
> *) mod_md:
> - Domain names in `` can now appear in quoted form.
> - Fixed a failure in ACME challenge selection that aborted further searches
> when the tls-alpn-01 method did not seem to be suitable.
> - Changed the tls-alpn-01 setup to only become unsuitable when none of the
> dns names showed support for a configured 'Protocols ... acme-tls/1'. This
> allows use of tls-alpn-01 for dns names that are not mapped to a VirtualHost.
> 
> 
> 
> git-svn-id: 
> https://svn.apache.org/repos/asf/httpd/httpd/branches/2.4.x@189168513f79535-47bb-0310-9956-ffa450edef68
> Want to know about upcoming build environment updates?
> 
> Would you like to stay up-to-date with the upcoming Travis CI build 
> environment updates? We set up a mailing list for you!
> 
> SIGN UP HERE
>  Documentation about Travis CI
> Have any questions? We're here to help.
> Unsubscribe from build emails from the apache/httpd repository.
> To unsubscribe from all build emails, please update your settings.
> 
> Travis CI GmbH, Rigaer Str. 8, 10427 Berlin, Germany | GF/CEO: Randy Jacops | 
> Contact: cont...@travis-ci.com | Amtsgericht Charlottenburg, Berlin, HRB 
> 140133 B | Umsatzsteuer-ID gemäß §27 a Umsatzsteuergesetz: DE282002648



Re: Broken: apache/httpd#1754 (2.4.x - 0244d12)

2021-07-20 Thread Ruediger Pluem
Missing backport of r1891477, r1891591?

Regards

Rüdiger

On 7/20/21 4:56 PM, Travis CI wrote:
> apache
> 
> /
> 
> httpd
> 
> 
> 
> branch icon2.4.x 
> 
> build has failed
> Build #1754 was broken 
> 
> arrow to build time
> clock icon25 mins and 20 secs
> 
> Stefan Eissing avatarStefan Eissing
> 
> 0244d12 CHANGESET → 
> 
> 
> Merged /httpd/httpd/trunk:r1891683
> 
> *) mod_md:
> - Domain names in `` can now appear in quoted form.
> - Fixed a failure in ACME challenge selection that aborted further searches
> when the tls-alpn-01 method did not seem to be suitable.
> - Changed the tls-alpn-01 setup to only become unsuitable when none of the
> dns names showed support for a configured 'Protocols ... acme-tls/1'. This
> allows use of tls-alpn-01 for dns names that are not mapped to a VirtualHost.
> 
> 
> 
> git-svn-id: 
> https://svn.apache.org/repos/asf/httpd/httpd/branches/2.4.x@1891685 
> 13f79535-47bb-0310-9956-ffa450edef68
> 
> Want to know about upcoming build environment updates?
> 
> Would you like to stay up-to-date with the upcoming Travis CI build 
> environment updates? We set up a mailing list for you!
> 
> SIGN UP HERE 
> 
> book icon
> 
> Documentation  about Travis CI
> 
> Have any questions? We're here to help. 
> Unsubscribe
> 
>  from
> build emails from the apache/httpd repository.
> To unsubscribe from *all* build emails, please update your settings
> .
> black and white travis ci logo 
> 
> Travis CI GmbH, Rigaer Str. 8, 10427 Berlin, Germany | GF/CEO: Randy Jacops | 
> Contact: cont...@travis-ci.com
>  | Amtsgericht Charlottenburg, Berlin, HRB 
> 140133 B | Umsatzsteuer-ID gemäß §27 a
> Umsatzsteuergesetz: DE282002648
>