Michael Holzt wrote:
I've downloaded the checkbasicheaders plugin from svn and found it to be
non working. While analyzing the situation i noticed, that the plugin is
missing a call of $self->register_hook. I checked another plugin which was missing this as well. But yet another plugin still had the call. In
README.plugins the call is still described.

You cannot use check_basicheaders from svn without the matching Qpsmtpd::Plugins. You also didn't mention whether you pulled from trunk or branches/0.3x (the latter is where active development is occurring).

The short answer is that, as long as you name your sub the same as one of the hook values, Qpsmtpd::Plugins will install the hook automatically. If you don't want to do that, you are still free to use $self->register_hook. Not all of the plugins in the repository have been altered to use the new modality, and some cannot use this mode. In particular, auth/auth_vpopmail_mysql uses the same sub to hook multiple auth hooks, so it will always use register_hook().

John

Reply via email to