On Fri, Aug 04, 2017 at 02:56:21PM +0200, Gilles Chehade wrote: > On Fri, Aug 04, 2017 at 01:13:06PM +0200, Eric Faurot wrote: > > Hi, > > > > Experimental support for filters has been removed some time ago from > > the config parser. Now we want to get rid of the remaining code. > > It's not that trivial, so we proceed in several steps. > > > > The first (and trickiest) one is to bypass the filter code for > > incoming smtp sessions, so that filter.c can be unhooked. > > This is what the following diff does: > > > > - drop filter configuration, > > - drop filter events, > > - simulate a positive reply for all filter queries, > > - write message content directly to the file. > > > > There should be no functionnal change. > > > > this should be tested by many people right away to spot subtle regressions >
Just a clarification because i've received the same question three times since this mail :-) We're not killing filters, most of the implementation is correct and the filters support is pretty much functional at the exception of some cases which are very unfortunately show stoppers... We're removing the filter code from the SMTP state machine to plug it at a different spot and fix the current limitations. Most of the code is to be reused, this is not a rewrite but a refactor. -- Gilles Chehade https://www.poolp.org @poolpOrg