Hi all.

I've uploaded three plugin tests and supporting files to the tracker:
http://code.google.com/p/smtpd/issues/list

as issue 14.  The upload is a tar file containing:

        gwh-07Mar26/t/plugin_tests/auth::authdeny
        gwh-07Mar26/t/plugin_tests/auth::authnull
        gwh-07Mar26/t/plugin_tests/auth::auth_flat_file
        gwh-07Mar26/config.sample/flat_auth_pw
        gwh-07Mar26/config.sample/plugins

and if these are copied into the current repo (r728) then the
appropriate plugins (auth:: -> auth/) are tested (and pass the
tests ;-).

However, it is probably not appropriate to list auth/authnull and
auth/authdeny in config.sample/plugins as they are deprecated by the
author viz:

#  This plugin doesn't actually check anything and will authenticate any
#  user no matter what they type.  It is strictly a proof of concept for
#  the Qpsmtpd::Auth module.  Don't run this in production!!!

I would like to propose that we create 'config.tests' and change the
code in t/Test/Qpsmptd.pm from:
        sub config_dir {
            './config.sample';
        }
to:
        sub config_dir {
            './config.tests';
        }

I will upload the appropriate svn-diff as a patch to the tracker if this
change is adopted (my previous plugin-test added some other cruft to
config.sample, which should be removed as well).

I will need to see what gets done with the files above before I can
really proceed efficiently ... so I am going to occupy myself by writing
a few more tests (and a plugin :-) and play with swaks and procmail
while I wait.

tqvm

-- 
--gh


Reply via email to