Closed
Bug 310115
Opened 20 years ago
Closed 20 years ago
Phishing detection no longer works
Categories
(Thunderbird :: Mail Window Front End, defect)
Thunderbird
Mail Window Front End
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 310068
People
(Reporter: iannbugzilla, Assigned: mscott)
References
Details
(Keywords: regression)
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.
Assignee | ||
Comment 1•20 years ago
|
||
branch or trunk ian?
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.
Assignee | ||
Comment 4•20 years ago
|
||
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•20 years ago
|
||
*** This bug has been marked as a duplicate of 310068 ***
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•