On Sat, 9 Jul 2005, Charlie Brady wrote: > > On Fri, 8 Jul 2005, Matt Sergeant wrote: > > > Can you try the attached plugin, along with *just* your change to reset the > > connection. I *think* that's the only thing we need to change to get this > > to > > work as a plugin rather than changing the core (again). And I'd much rather > > we could do this via the plugin API (same as I would for auth, but that's > > another argument :-) > > OK, I've taken the path of least resistence and switched to svn trunk for > testing. > > I've found two small problems so far with using the unrecognised command > hook to implement ESMTP. First is that any params don't seem to get passed > to the callback. The "Syntax error (no parameters allowed)" check isn't > triggered:
Ah yes I patched that locally. > Second problem is that every extended verb will contribute to the > "unrocognized command" count, possibly causing a disconnect: Nope - if you implement an extended verb you don't return DECLINED, and thus count_unrecognised_commands doesn't get run. This does rely on plugin ordering of course.
