[ dspam-Bug Tracker-3000563 ] Quarantine won't deliver mail

Bug Tracker item #3000563, was opened at 2010-05-12 18:53
Message generated for change (Comment added) made by sbajic
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=1126467&aid=3000563&group_id=250683

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: webui
>Group: v3.6.8
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Glen Thorne (glenthorne)
>Assigned to: Stevan Bajic (sbajic)
Summary: Quarantine won't deliver mail

Initial Comment:
When using virtual users (default data storage at /usr/local/var/dspam/data), the <username>.mbox file gets created with either a simple NL or a CR+NL end of line marker.  This causes the dspam.cgi script to not identify messages properly and fails to process the mbox file.  I have added a few lines of simple code to remove the CRs in the lines and this has resolved the problem.  I will upload the file and you can run a diff on it to see the change.

----------------------------------------------------------------------

>Comment By: Stevan Bajic (sbajic)
Date: 2010-08-17 19:17

Message:
Hallo Glen,

thanks for reporting this issue. Looking at the file you attached I see
that it must be from a old DSPAM release. Right? Can you tell me what
version of DSPAM you got that dspam.cgi from?

In your header I see:
# $Id: dspam.cgi,v 1.26 2006/06/01 19:14:14 jonz Exp $

While our current dspam.cgi has:
# $Id: dspam.cgi,v 1.55 2010/08/03 12:52:06 sbajic Exp $

You can get the latest DSPAM web-UI from GIT or using HTTP and the
following link:
http://dspam.git.sourceforge.net/git/gitweb.cgi?p=dspam/dspam;a=tree;f=webui;hb=HEAD

I don't know if the error with the quarantine MBOX is fixed with the
newest edition of the web-UI? It's hard to see what you have changed when I
don't know what version you used as base for your changes. I think you
added "chomp" to ProcessFalsePositive, Quarantine_ManyNotSpam,
Quarantine_ViewMessage, Quarantine_DeleteSpam and DisplayQuarantine. And
you added a "s/\r//g;" to CheckQuarantine. Can this be? Could you not post
a diff file or attach the original file too so that I can compare what you
have added?

Would be great if you could try the Web-UI from 3.9.1 or from GIT and tell
me if you have the same issue with the new Web-UI?


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=1126467&aid=3000563&group_id=250683

------------------------------------------------------------------------------
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