310115 - Phishing detection no longer works

Status: RESOLVED DUPLICATE of bug 310068

(Thunderbird :: Mail Window Front End, defect)

Description

[Ian Neal]

Build ID 2005092505 or later
Steps to reproduce
1. Generate an email with a known phishing trigger
2. Open it in Thunderbird or mailnews

Expected result
1. Message bar shows it as a scam/phishy

Actual result
1. Message bar does not change

Works fine in BuildID 2005092405 or earlier

Using JS debugger with mailnews seems to show the for loop

var anchorNodes = msgDocument.getElementsByTagName("a");
for (var index = 0; index < anchorNodes.length && !isEmailScam; index++)
  isEmailScam = isPhishingURL(anchorNodes[index], true);

not getting past index = 0 with isEmailScam being false

only checkins in that period seem to be do with DOM: Core - nothing TB/mailnews
related.

Comment 1

[Scott MacGregor]

branch or trunk ian?

Comment 2

[Ian Neal]

trunk

Comment 3

[Ian Neal]

The other alternative is /be's jsinterp.c changes which have had some regression
fixes put in, but I've just tested
http://ftp.mozilla.org/pub/mozilla.org/thunderbird/tinderbox-builds/patrocles-trunk/
which should have them in and it still does not work.

Comment 4

[Scott MacGregor]

anchorNodes.length is coming back as undefined for me even though the document
clearly has several anchor elements.

I think this call isn't working:

var msgDocument = document.getElementById('messagepane').contentDocument;
var anchorNodes = msgDocument.getElementsByTagName("a");

Comment 5

[Peter Van der Beken [:peterv]]

*** This bug has been marked as a duplicate of 310068 ***