Don't mark as scam any mail composed by user (Drafts, Templates, Sent, Outbox folders)

RESOLVED FIXED in Thunderbird 3.0a3

Status

defect
RESOLVED FIXED
14 years ago
4 years ago

People

(Reporter: matjaz.bugzilla.2005, Assigned: mkmelin)

Tracking

unspecified
Thunderbird 3.0a3

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7.10) Gecko/20050716 Firefox/1.0.6
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7.10) Gecko/20050716 Firefox/1.0.6

Saving an email as a draft results in TB marking it as possible scam if it
contains a link to a any site written as IP address instead of domain name. This
is also the case if the linki is local intranet address.

Reproducible: Always

Steps to Reproduce:
1. Create a new email.
2. Enter any address as IP: http://10.12.25.99
3. Save it
4. Open this email in drafts folder

Actual Results:  
Thunderbird thinks this email is scam

Expected Results:  
Email messages that are composed by a users himself cannot be scams.

Note: TB also marks as possible scams messages that contain links to local
intranet sites. Such messages should not be marked as possible scams, even if
they are received messages.
Forgot to mention> I'm using TB version 1.0+ (20050817)
Component: Security → Mail Window Front End
Summary: Mail in drafts marked as possible scam (adrress with IP in body) → Mail in drafts marked as possible scam (address with IP in body)
confrming.

changing summary to make it more general:
was:  Mail in drafts marked as possible scam (address with IP in body)
now:  Don't mark as scam any mail composed by user (Drafts, Sent folders)


In my TB 1.5, it marks e-mails in my Sent folder as Scams.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Summary: Mail in drafts marked as possible scam (address with IP in body) → Don't mark as scam any mail composed by user (Drafts, Sent folders)
*** Bug 337612 has been marked as a duplicate of this bug. ***
on the other hand, fixing this would make it harder to test what will be detected as scam and what won't. bug 308366 and bug 328455 deal with the (local) ip address issue.
I'm not sure this is really a bug for dveditz.

I think that tbird already special cases certain actions for certain folders (like don't check messages in your trash to see if they are junk.  see http://lxr.mozilla.org/mozilla1.8/source/mailnews/base/util/nsMsgDBFolder.cpp#1892)

scott, perhaps tbird can do the same with the scam detection?
See Bug 308366 which made it so we don't mark local IP addresses as scams.
Came to me because originally filed as a security bug. -> default assignee
Assignee: dveditz → mscott
QA Contact: thunderbird → front-end
OS: Windows XP → All
Hardware: PC → All
*** Bug 351787 has been marked as a duplicate of this bug. ***
I'm not seeing this happening on Trunk, Shredder.  Can anyone else confirm?
Assignee: mscott → nobody
No wonder, the scam detector was broken - bug 452661.
Assignee: nobody → mkmelin+mozilla
Summary: Don't mark as scam any mail composed by user (Drafts, Sent folders) → Don't mark as scam any mail composed by user (Drafts, Templates, Sent, Outbox folders)
Posted patch proposed fix (obsolete) — Splinter Review
Attachment #335945 - Flags: review?(philringnalda)
Status: NEW → ASSIGNED
Target Milestone: --- → Thunderbird 3.0b1
Please "borrow" the comment from SeaMonkey's copy of this file about how aUrl.folder will throw for a saved .eml - it took me quite a bit of poking to figure out why we were hiding this all away in a try/catch.

And since we don't want to silently swallow any other exceptions, can we test ex, and rethrow if it's anything other than that one expected one? (I didn't look yet at how unhappy IsSpecialFolder is with a null folder, to see if I actually mean "those expected two.")
Added the comment and checking we don't hide other than the expected error.

IsSpecialFolder is quite happy with a null folder.
Attachment #335945 - Attachment is obsolete: true
Attachment #337524 - Flags: review?(philringnalda)
Attachment #335945 - Flags: review?(philringnalda)
Comment on attachment 337524 [details] [diff] [review]
proposed fix, v2

Looks good to me, thanks!
Attachment #337524 - Flags: review?(philringnalda) → review+
changeset:   307:55c2a5ba26b6
http://hg.mozilla.org/comm-central/rev/55c2a5ba26b6

->FIXED
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
This issue appears to have resurfaced.

Replicate

1. Compose a new message to a friend, such as:

From https://www.upwork.com/o/jobs/job/_~014308dd0e75d018fd/ :

https://s3.amazonaws.com/stuffpub/brief-charcter-rig-1.swf
https://s3.amazonaws.com/stuffpub/brief-charcter-rig-2.swf

2. Click "Send".
3. Click "Send Mail".

Expected Results

Message is not marked as a possible scam.

Actual Results

Message is marked as a possible scam.
Sorry, step 3 should be:

3. Click "Sent Mail".

Using Thunderbird 31.7.0 on Xubuntu Linux.
You need to log in before you can comment on or make changes to this bug.