User-Agent: Mozilla/5.0 (X11; U; Linux i686 (x86_64); en-US; rv:126.96.36.199) Gecko/20080201 Firefox/188.8.131.52 Build Identifier: Mozilla/5.0 (X11; U; Linux i686 (x86_64); en-US; rv:184.108.40.206) Gecko/20080201 Firefox/220.127.116.11 If the phishing server returns 400's, Firefox will make the request every minute, instead of just making 3 requests, and then backing off for an hour. Reproducible: Always Steps to Reproduce: 1. Have server return 400.
Bumping up the priority -- we really need to get this fixed for beta 5, so that Firefox won't hammer the phishing servers if it receives 400's.
The backoff object doesn't consider 400 to be an error. This should be an easy fix, I'll attach a patch soon.
Created attachment 310401 [details] [diff] [review] trigger backoff for 400s
Created attachment 310410 [details] [diff] [review] slightly better fix This version avoids scheduling the one-minute reping if the backoff object doesn't consider the server response worth erroring out on.
Comment on attachment 310410 [details] [diff] [review] slightly better fix This is fine, but it's also a dupe of bug 378132. This fix seems to be slightly better since we're now considering 4xx an error (that's not how we spec'ed it originally).
Checking in content/listmanager.js; /cvsroot/mozilla/toolkit/components/url-classifier/content/listmanager.js,v <-- listmanager.js new revision: 1.28; previous revision: 1.27 done Checking in content/request-backoff.js; /cvsroot/mozilla/toolkit/components/url-classifier/content/request-backoff.js,v <-- request-backoff.js new revision: 1.2; previous revision: 1.1 done
The patch in bug 378132 might be a safer basis for a branch patch, as it changes fewer interfaces (a bit). You'd want to update it to treat 4xx as errors.
Clearing branch blocking request because we took the patch in bug 378132. If we still need more please re-nominate and explain.
Dave/Garret, can you verify this bug is fixed, i think its difficult for QA to generate this Error 400 :-)