> So, I've spent some time on debugging and I think I found the bug. Maybe > your patch addresses the same issue (the response from the quota module > is not cleared when the quota is not reached), but unfortunately it did > not work. The attached diff solved my problem -- if verdict is not set, > I explicitly set the (in my opinion) correct protocol_response.
This would unfortunately nuke any response another module has set. What I did fix is adding DEFER to the Quotas module, but seeing as this wasn't in there before, I cannot imaging how it was working. Try remove the text in your data field in your quota definition, v2 should set this to a more terse reason which may lead to the part of code its coming from. > Maybe you find a better solution for this issue... Attached the addition of DEFER. Failing it still doesn't work, paste your current DB entries and a full log of the above with all debugging options, I'll take a look and see if I can't see anything. -N
Index: cbp/modules/Quotas.pm
===================================================================
--- cbp/modules/Quotas.pm (revision 261)
+++ cbp/modules/Quotas.pm (working copy)
@@ -459,6 +459,8 @@
$sessionData->{'Recipient'});
return CBP_CONTINUE;
+ } if ($verdict =~ /^defer$/i) {
+ return $server->protocol_response(PROTO_DEFER,$verdict_data);
} if ($verdict =~ /^hold$/i) {
return $server->protocol_response(PROTO_HOLD,$verdict_data);
} elsif ($verdict =~ /^reject$/i) {
Index: cbp/modules/Amavis.pm
===================================================================
signature.asc
Description: This is a digitally signed message part
------------------------------------------------------------------------- This SF.Net email is sponsored by the Moblin Your Move Developer's challenge Build the coolest Linux based applications with Moblin SDK & win great prizes Grand prize is a trip for two to an Open Source event anywhere in the world http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________ policyd-users mailing list policyd-users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/policyd-users
