Closed
Bug 1198443
Opened 9 years ago
Closed 7 years ago
Downloaded digest256 entries are always considered stale
Categories
(Toolkit :: Safe Browsing, defect, P3)
Toolkit
Safe Browsing
Tracking
()
RESOLVED
INVALID
People
(Reporter: francois, Unassigned)
References
(Depends on 1 open bug)
Details
Attachments
(1 obsolete file)
Steps:
1. enable TP and ensure that the list has just been updated
2. take mozpub-track-digest256 out of disallow_completions
3. visit https://people.mozilla.org/~fmarier/tracking-test/
Expected:
The list was just downloaded, all of the entries in it should be considered fresh and honored.
Actual:
The list appears to be stale and its entries are not honored because the gethash call fails.
-1589647616[7fcca8883160]: Complete in mozpub-track-digest256
-1589647616[7fcca8883160]: Found a result in mozpub-track-digest256: complete. (Age: 86400s)
-1589647616[7fcca8883160]: Found 1 results.
-1589647616[7fcca8883160]: Found 1 results.
-1589647616[7fcca8883160]: query took 0ms
Reporter | ||
Updated•8 years ago
|
Priority: -- → P5
Reporter | ||
Updated•8 years ago
|
Priority: P5 → P3
Updated•8 years ago
|
Assignee: nobody → tnguyen
Comment hidden (obsolete) |
Comment hidden (obsolete) |
Comment 3•8 years ago
|
||
From what we have in comment 1, somehow, the fresh value was not stored after doing update (although completion was stored). I still have no idea that, probably there's update failure or something like that.
I tried to test several times manually, it seems ok, will try to make more tests and look closer.
Reporter | ||
Comment 4•8 years ago
|
||
Un-assigning for now since I don't think Thomas is actively working on it.
Assignee: tnguyen → nobody
Reporter | ||
Comment 5•7 years ago
|
||
This may be fixed now that we are using the V4 negative cache for V2 lists too.
Reporter | ||
Comment 6•7 years ago
|
||
Actually, this is how the protocol is supposed to work according to Google. Full hashes in V2 still need to be confirmed via a gethash call before we are allowed to use them to block a website.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•