756821 - Port |Bug 305089: Don't mark as scam any mail composed by user (Drafts, Templates, Sent, Outbox folders)| and non-IPv6 parts of |Bug 362434: Add IPv6 support to phishingDetector.js|

Status: RESOLVED FIXED

(SeaMonkey :: MailNews: Message Display, defect)

Description

[Ian Neal]

Attachment: Ported parts

Port the following from TB to SM:
Bug 305089: Don't mark as scam any mail composed by user (Drafts, Templates, Sent, Outbox folders)
and the relevant parts of:
Bug 436044 - Convert some more nsMsgFolderFlags users over to the id
Bug 506577: convert users of IsSpecialFolder(folder, ...) over to use folder.isSpecialFolder(...)
The non-IPv6 parts of:
Bug 362434: Add IPv6 support to phishingDetector.js

The IPv6 parts are covered in bug 80855

Comment 1

[[e-mail]]

Comment on attachment 625442 [details] [diff] [review]
Ported parts

I assume we're going to get the improved IP address detector at some point?

>+    var unobscuredHostName = hostNameIsIPAddress(hrefURL.host);
Might want to rename unobscuredHostName to ipAddress

>-      isPhishingURL = confirmSuspiciousURL(phishingType, unobscuredHostName.value);
>+      isPhishingURL = confirmSuspiciousURL(phishingType, unobscuredHostName);
You're passing in the IP address even when it's not that phishing type, and therefore it is empty. I'm not sure passing the detected IP address is useful even when it exists, you should probably always pass the original host.

>+// returns the unobscured host (if there is one), otherwise false
Might want to return null on failure

Comment 2

[Ian Neal]

Attachment: Updated for review comments [Checked in: Comment 3, 4 & 5]

Carrying forward r=

Comment 3

[Ian Neal]

Comment on attachment 625525 [details] [diff] [review]
Updated for review comments [Checked in: Comment 3, 4 & 5]

http://hg.mozilla.org/comm-central/rev/58044311b637

[Approval Request Comment]
Regression caused by (bug #): 
User impact if declined: Less robust checking of scam emails
Testing completed (on m-c, etc.): Landed on trunk
Risk to taking this patch (and alternatives if risky): Low risk
String changes made by this patch: None

Comment 4

[Philip Chee]

Pushed to comm-beta.
http://hg.mozilla.org/releases/comm-beta/rev/f7430f8d47df

Comment 5

[Ian Neal]

Comment on attachment 625525 [details] [diff] [review]
Updated for review comments [Checked in: Comment 3, 4 & 5]

http://hg.mozilla.org/releases/comm-aurora/rev/11549db5edf6