All users were logged out of Bugzilla on October 13th, 2018
The url classifier stream updater doesn't check the http status of the update request. It should verify status 200 before trying to update (not a big deal since it ignores malformed lines).
Created attachment 229007 [details] [diff] [review] v1: check RequestSucceeded before updateing
Attachment #229007 - Flags: review?(darin)
Target Milestone: --- → Firefox 2 beta2
Tony, did you intentionally comment out the LOG call? Not clear how that's related here.
Status: NEW → ASSIGNED
Flags: blocking-firefox2? → blocking-firefox2+
(In reply to comment #3) > Tony, did you intentionally comment out the LOG call? Not clear how that's > related here. Yes, it was really verbose, even for debugging.
Comment on attachment 229007 [details] [diff] [review] v1: check RequestSucceeded before updateing a=drivers, please land on the branch.
Attachment #229007 - Flags: approval1.8.1? → approval1.8.1+
Status: ASSIGNED → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
Component: Phishing Protection → Phishing Protection
Product: Firefox → Toolkit
You need to log in before you can comment on or make changes to this bug.