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: Pending
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-22 20:12

Message:
Hallo Glen, had you the chance to test if the GIT version still has this
issue? Since I expect this to be an issue with the old 3.6.8 edition of
DSPAM and since we have made so many changes after 3.8.0 I expect this
issue to not be present in 3.9.0/3.9.1 or latest GIT. I am now going to set
the status of this bug report to pending. Please respond in two weeks or
this bug report is going to be automatically closed by the Sourceforge bug
tracking system.

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

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 
_______________________________________________
Dspam-devel mailing list
Dspam-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dspam-devel

Reply via email to