On Sun, 08 Aug 2010 15:52:58 +0200
Tom Hendrikx <t...@whyscream.net> wrote:

[...]
> > 
> > # cat /var/spool/dspam/group.away
> > groupname:classification:*classicifation_user
> > 
> >> Could you post the output of "select * from dspam_virtual_uids where 
> >> username = 'classification_user'"?
> > 
> > vmail=# select * from dspam_virtual_uids;
> >  uid |        username
> > -----+------------------------
> >    1 | classification_user
> >    2 | tomhe...@example.com
> > (2 rows)
> > 
> 
> And a few minutes after sending the mail, I thought that I might have
> made a typo. And I did: classification_user != classicifation_user
> 
Good catch.


> Sorry for the noise. However, I still think that an error message
> explaining the issue would help a lot.
> 
> Error: processing message for user tomhe...@example.com failed: unable
> to find user 'classicifation_user' for peer classification.
> 
> looks so much better than:
> 
> bailing on error -2
> process_message returned error -5.  delivering.
> 
You are right. The error message could be better. However... it's not that 
simple. I anyway need to change the whole code for better error handling in 
case of non existing users.

Could you try the attached patch? It will not print out the error message you 
requested but should work better in cases when ensure_confident_result() is 
returning a wrong result.

Could you send me a new debug output with the applied patch?


> >> Do you have more lines from the log? I mean: Could you post more lines 
> >> from the log from above? How can 
> >> "_pgsql_drv_getpwnam(classicifation_user)" return NULL if the user 
> >> exists/existed in dspam_virtual_uids?
> >>
> > 
> > I created a new debug file, see attachment for complete output.
> > 
> > 
> > 
> > 
> > ------------------------------------------------------------------------------
> > This SF.net email is sponsored by 
> > 
> > Make an app they can't live without
> > Enter the BlackBerry Developer Challenge
> > http://p.sf.net/sfu/RIM-dev2dev 
> > 
> > 
> > 
> > _______________________________________________
> > Dspam-devel mailing list
> > Dspam-devel@lists.sourceforge.net
> > https://lists.sourceforge.net/lists/listinfo/dspam-devel
> 
> 
> -- 
> Regards,
>       Tom
> 

Attachment: tom.patch
Description: Binary data

------------------------------------------------------------------------------
This SF.net email is sponsored by 

Make an app they can't live without
Enter the BlackBerry Developer Challenge
http://p.sf.net/sfu/RIM-dev2dev 
_______________________________________________
Dspam-devel mailing list
Dspam-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dspam-devel

Reply via email to