836769 - Hook up MailNews feeds display to the Safe Browsing code so that the buttons and notifications work.

Status: NEW

(SeaMonkey :: MailNews: General, defect)

Description

[Philip Chee]

From Bug 477718 Comment 20:

> I also tried the URLs in a faked feed in MailNews. The warnings appeared, 
> but not too surprisingly none of the buttons worked (nothing on the Error 
> Console). If this is an issue, it can surely be moved into a follow-up; I 
> just included this for the sake of completeness.

Comment 1

[rsx11m]

Just for clarification, is this about RSS feeds or also phishy newsgroup feeds?

Comment 2

[Philip Chee]

I think this is about the RSS feeds (newsblog) in SeaMonkey MailNews.

A more complicated issue is to make our suite/mailnews/phishingDetector.js also check the phish/malware tables when any mail message containing dubious links is viewed. At the moment our phishingDetector.js just does static checks, I think.

Comment 3

[Philip Chee]

> I think this is about the RSS feeds (newsblog) in SeaMonkey MailNews.
This would probably be simpler once Bug 633937 is fixed.

q.v. Bug 633937 - Make use of the toolkit's notificationbox to display notifications in MailNews (Port Bug 562048)

Comment 4

[rsx11m]

(In reply to Philip Chee from comment #2)
> A more complicated issue is to make our suite/mailnews/phishingDetector.js also
> check the phish/malware tables when any mail message containing dubious links is
> viewed. At the moment our phishingDetector.js just does static checks, I think.

Yes, it's just a fixed set of rules (see Thunderbird meta bug 654502). Making it use the tables to improve the scam warning certainly would be a plus. On the other hand, one may argue that it should be sufficient already to show the browser's warning when clicking on a link. Anyway, that's a discussion for a new bug then.

Comment 5

[rsx11m]

Filed bug 837386 for the message display.