Minor bug in plugins/check_badmailfrom

2009-06-21 Thread Tomas Lee
I found a minor bug in plugins/check_badmailfrom, and I didn't know where to send it, so I'm posting it here. When a badmailfrom issues the "RCPT TO" command, a 550 error code is sent with the reason it was rejected. The default reason is supposed to be "sorry, your envelope sender is in my badma

Mis-documentation in README: check_relay vs. rcpt_ok

2009-06-21 Thread Tomas Lee
The README file says: At least one plugin MUST allow or deny the RCPT command to enable receiving mail. The "check_relay" plugin is the standard plugin for this. But in version 0.29, the "check_relay" plugin split into two plugins: "check_relay" and "rcpt_ok". "rcpt_ok" is the one t

Mis-documentation in docs/config.pod, spool_dir

2009-06-21 Thread Tomas Lee
docs/config.pod says: spool_dir Where temporary files are stored, defaults to tmp/. FIXME: is this correct? That's not correct. It should be "~/tmp/", or as the README puts it, "$ENV{HOME}/tmp/".

Re: Mis-documentation in README: check_relay vs. rcpt_ok

2009-06-21 Thread Ask Bjørn Hansen
On Jun 21, 2009, at 14:11, Tomas Lee wrote: But in version 0.29, the "check_relay" plugin split into two plugins: "check_relay" and "rcpt_ok". "rcpt_ok" is the one that actually hooks into RCPT now, so shouldn't the README file say "rcpt_ok" instead of "check_relay"? Thanks; that's almost 5

Re: Mis-documentation in docs/config.pod, spool_dir

2009-06-21 Thread Ask Bjørn Hansen
On Jun 21, 2009, at 16:40, Tomas Lee wrote: docs/config.pod says: spool_dir Where temporary files are stored, defaults to tmp/. FIXME: is this correct? That's not correct. It should be "~/tmp/", or as the README puts it, "$ENV{HOME}/tmp/". Yup. Updated docs/config.pod