Open Bug 836769 Opened 8 years ago Updated 3 years ago

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

Categories

(SeaMonkey :: MailNews: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

People

(Reporter: philip.chee, Unassigned)

References

Details

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.
Just for clarification, is this about RSS feeds or also phishy newsgroup feeds?
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.
> 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)
Depends on: 633937
(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.
Filed bug 837386 for the message display.
See Also: → 778611
You need to log in before you can comment on or make changes to this bug.