Closed
Bug 310115
Opened 19 years ago
Closed 19 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•19 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•19 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•19 years ago
|
||
*** This bug has been marked as a duplicate of 310068 ***
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•