395377 - query string update to the safebrowsing lookup algorithm

Status: RESOLVED FIXED

(Toolkit :: Safe Browsing, defect)

Description

[Dave Camp (:dcamp)]

Attachment: update for new safebrowsing lookup changes

Since the implementation of the new safebrowsing protocol, there have been two changes to the spec:

a) The URI is supposed to be checked with all query parameters intact.
b) The URI-with-a-$ part has been removed.

Comment 1

[Tony Chang (Google)]

Comment on attachment 280063 [details] [diff] [review]
update for new safebrowsing lookup changes

Looks good.

Comment 2

[Dave Camp (:dcamp)]

Checking in toolkit/components/url-classifier/src/nsUrlClassifierDBService.cpp;
/cvsroot/mozilla/toolkit/components/url-classifier/src/nsUrlClassifierDBService.cpp,v  <--  nsUrlClassifierDBService.cpp
new revision: 1.31; previous revision: 1.30
done
Checking in toolkit/components/url-classifier/src/nsUrlClassifierUtils.cpp;
/cvsroot/mozilla/toolkit/components/url-classifier/src/nsUrlClassifierUtils.cpp,v  <--  nsUrlClassifierUtils.cpp
new revision: 1.8; previous revision: 1.7
done
Checking in toolkit/components/url-classifier/tests/unit/test_dbservice.js;
/cvsroot/mozilla/toolkit/components/url-classifier/tests/unit/test_dbservice.js,v  <--  test_dbservice.js
new revision: 1.4; previous revision: 1.3
done