Matt Sergeant said the following on 02/23/2006 09:52 PM:
> On 23-Feb-06, at 4:27 PM, Robin Bowes wrote:
>
>> My preferred approach would be to apply the following simple patch to
>> qmail-queue to return DECLINED instead of OK following successful
>> queueing of the msg:
>>
>> --- qmail-queue 2006-02-23 02:04:42.000000000 +0000
>> +++ qmail-queue.patched 2006-02-23 21:23:43.000000000 +0000
>> @@ -76,7 +76,7 @@
>> my $msg_id = $transaction->header->get('Message-Id') || '';
>> $msg_id =~ s/[\r\n].*//s; # don't allow newlines in the Message-Id
>> here
>> $msg_id = "<$msg_id>" unless $msg_id =~ /^<.*>$/; # surround in
>> <>'s
>> - return (OK, "Queued! " . time . " qp $child $msg_id");
>> + return (DECLINED, "Queued! " . time . " qp $child $msg_id");
>> }
>> elsif (defined $child) {
>> # Child
>>
>> This would allow other plugins hooking queue to execute after the
>> qmail-queue plugin.
>>
>> However, I'm not sure if this has any negative impact in other areas of
>> qpsmtpd.
>>
>> Does anyone know if this will cause any problems?
>
>
> Yes. One queue plugin has to return OK or qpsmtpd responds with "no
> queue plugin worked" or something to that effect.
>
> I only list one queue plugin.
Matt,
I've got this set up on a test instance running on port 2525 and that
doesn't seem to be the case.
Neither plugin (qmail-queue or accepted_log) is returning OK - both
return DECLINED - and mail is delivered and I see no errors.
Are you *sure* about your previous statement?
R>
