Closed
Bug 1150334
Opened 10 years ago
Closed 10 years ago
Fragmentation in Safe Browsing chunks leads to 413 (Request Entity Too Large) during list updates
Categories
(Toolkit :: Safe Browsing, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: francois, Unassigned)
References
Details
Attachments
(2 files)
Monica started looking into bug 1147212 and noticed that she would sometimes get HTTP response 413 (Request Entity Too Large) in response to adding goog-unwanted-shavar. Sample request body is attached.
That's a very fragmented chunk space. It could be a regression on our end.
Reporter | ||
Comment 1•10 years ago
|
||
Comment 2•10 years ago
|
||
This is a log taken from within ProtocolParser showing what the server sends us the very first update.
You can see for example that sub chunk number 47176 is missing, although 47175 and 47177 are there. (see first 3 lines)
Adds chunks are continuous, sub chunks are all over the place.
This makes me suspect this may not be at our end.
Comment 3•10 years ago
|
||
I've done a build of Firefox ESR 10, which more or less uses the original SafeBrowsing implementation dating back to Firefox 3 (and likely dates back to Google's original implementation in their toolbar).
It has the same problem, i.e. it ends up with a totally fragemented subchunk list.
I conclude this is not a regression in our code.
Comment 4•10 years ago
|
||
Our alerts did pick up that something happened to the distribution of SafeBrowsing data at Firefox users:
http://alerts.telemetry.mozilla.org/index.html#/detectors/1/metrics/244/alerts/?from=2015-02-26&to=2015-02-26
Comment 5•10 years ago
|
||
Google's team pointed out in email conversation that the missing data is in the first redirect URL of the server responses. Looking in that direction now.
Comment 6•10 years ago
|
||
Further investigation didn't turn up much interesting. WireShark confirms that the list of redirects we process matches what the server sends back.
I can demonstrate that the list is fragmented via curl + grep, so I think that is more confirmation this is likely not our regression.
Reporter | ||
Comment 7•10 years ago
|
||
The request size limit on the server will be increased from 20KB to 64KB sometime this week or early
next week.
Reporter | ||
Comment 8•10 years ago
|
||
The fix is apparently in production, so I'll assume this is fixed.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•