Closed Bug 1533884 Opened 2 years ago Closed 4 months ago

Add telemetry to measure file corrupted count detected by CRC32 and sanity check

Categories

(Toolkit :: Safe Browsing, task, P2)

task

Tracking

()

RESOLVED INVALID

People

(Reporter: dimi, Unassigned)

References

Details

The SHA256 integrity check for prefix files is replaced by CRC32 in Bug 1353956 for performance. We also added a simple sanity check while loading the prefix file.

It is worth adding telemetry to see how those methods help us find a corrupted prefix file. One thing we want to know from the telemetry is if the sanity check is good enough to detect corrupted file, if it is, we can also remove CRC32 from loading/writing prefixes.

Assignee: nobody → dlee
Status: NEW → ASSIGNED
Priority: P2 → P1
Type: enhancement → task
Priority: P1 → P2

Not actively work on this

Assignee: dlee → nobody
Status: ASSIGNED → NEW

I don't think we need this anymore

Status: NEW → RESOLVED
Closed: 4 months ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.