Re: async filters still borked (was Re: svn commit: r1874775 - /httpd/httpd/trunk/test/README.travis)
On 20 Mar 2020, at 15:46, Joe Orton wrote: >> Anyone found time to check >> >> https://github.com/apache/httpd/pull/88 >> >> regarding the async filter bug? > > [crickets] > > We are two months on since discussing this first. It seems quite > worrying that the async filters stuff was introduced with apparently > nasty bugs and yet is already bitrotting on trunk with not enough people > to review and fix it. Can we move the code to a branch until it is > ready? First time I've seen this thread - can you be more specific, what nasty bugs are you referring to? I see http://svn.apache.org/viewvc?view=revision=1874775 refers to "The async filter bug”, but there is no PR, or any indication of what the bug is? Regards, Graham — smime.p7s Description: S/MIME cryptographic signature
Re: async filters still borked (was Re: svn commit: r1874775 - /httpd/httpd/trunk/test/README.travis)
On Fri, Mar 20, 2020 at 2:46 PM Joe Orton wrote: > > [crickets] > > We are two months on since discussing this first. It seems quite > worrying that the async filters stuff was introduced with apparently > nasty bugs and yet is already bitrotting on trunk with not enough people > to review and fix it. Sorry for not being able to work on this so far, just proposed a new patch in the original thread. > Can we move the code to a branch until it is > ready? This probably wouldn't be easy, there are some further changes (on trunk) that depend on the async filters, like ap_proxy_transfer_between_connections() shared by several proxy modules now, plus mpm_event obviously. But if we can't make async filters work with reasonable changes, sure let's move it out of trunk. Regards, Yann.
Re: async filters still borked (was Re: svn commit: r1874775 - /httpd/httpd/trunk/test/README.travis)
On 3/20/20 2:46 PM, Joe Orton wrote: > On Fri, Mar 06, 2020 at 09:30:41AM +0100, Ruediger Pluem wrote: >> On 3/4/20 9:23 AM, jor...@apache.org wrote: >>> Author: jorton >>> Date: Wed Mar 4 08:23:55 2020 >>> New Revision: 1874775 >>> >>> URL: http://svn.apache.org/viewvc?rev=1874775=rev >>> Log: >>> Update docs. The expr_string.t failure has not been seen since >>> the workaround was added AFAICT. The async filter bug >>> is still breaking the tests regularly. [skip ci]. >> >> Anyone found time to check >> >> https://github.com/apache/httpd/pull/88 >> >> regarding the async filter bug? > > [crickets] > > We are two months on since discussing this first. It seems quite > worrying that the async filters stuff was introduced with apparently > nasty bugs and yet is already bitrotting on trunk with not enough people > to review and fix it. Can we move the code to a branch until it is > ready? Maybe better, yes. So +1. Regards Rüdiger
Re: async filters still borked (was Re: svn commit: r1874775 - /httpd/httpd/trunk/test/README.travis)
+1 from me. > On Mar 20, 2020, at 9:46 AM, Joe Orton wrote: > > On Fri, Mar 06, 2020 at 09:30:41AM +0100, Ruediger Pluem wrote: >> On 3/4/20 9:23 AM, jor...@apache.org wrote: >>> Author: jorton >>> Date: Wed Mar 4 08:23:55 2020 >>> New Revision: 1874775 >>> >>> URL: http://svn.apache.org/viewvc?rev=1874775=rev >>> Log: >>> Update docs. The expr_string.t failure has not been seen since >>> the workaround was added AFAICT. The async filter bug >>> is still breaking the tests regularly. [skip ci]. >> >> Anyone found time to check >> >> https://github.com/apache/httpd/pull/88 >> >> regarding the async filter bug? > > [crickets] > > We are two months on since discussing this first. It seems quite > worrying that the async filters stuff was introduced with apparently > nasty bugs and yet is already bitrotting on trunk with not enough people > to review and fix it. Can we move the code to a branch until it is > ready? > > Regards, Joe >