Tinderbox should discard messages that don't look like a client report instead of bouncing them

RESOLVED FIXED

Status

--
critical
RESOLVED FIXED
14 years ago
4 years ago

People

(Reporter: justdave, Assigned: cls)

Tracking

Details

Attachments

(1 attachment)

If an email contains expected Tinderbox headers, but those headers are missing
data or not all the correct headers are present, then the mail should bounce. 
If there are no headers at all, it should just be discarded.

The tinderbox-daemon address is starting to receive a lot of spam recently. 
Said spam of course bounces with an error stating that the tinderbox header was
incomplete.  Most of these bounces then postmaster-bounce on mecha because the
spammers of course used an invalid return address.
The pace of the spam has picked up a LOT recently.  For all of them that bounce,
I imagine a lot of them are going to innocent third-parties with real email
addresses that the spammers spoofed.  We really need to do something about this
or risk getting our tinderbox bounce notices reported as spam.
Severity: normal → critical
(Assignee)

Comment 2

13 years ago
Created attachment 221738 [details] [diff] [review]
v1.0
Assignee: mcafee → cls
Status: NEW → ASSIGNED
Attachment #221738 - Flags: review?(justdave)
(Assignee)

Updated

13 years ago
Attachment #221738 - Flags: review?(bear)

Comment 3

13 years ago
Comment on attachment 221738 [details] [diff] [review]
v1.0

I applied and ran the patch but do not have a live tinderbox setup to do the final test.
Attachment #221738 - Flags: review?(bear) → review+
(Assignee)

Updated

13 years ago
Attachment #221738 - Flags: review?(justdave)
(Assignee)

Comment 4

13 years ago
Checking in webtools/tinderbox/processbuild.pl;
/cvsroot/mozilla/webtools/tinderbox/processbuild.pl,v  <--  processbuild.pl
new revision: 1.48; previous revision: 1.47
done

Status: ASSIGNED → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
Product: Webtools → Webtools Graveyard
You need to log in before you can comment on or make changes to this bug.