wikimedia / restbase / 3768

84%

master: 84%

Pull #1185

travis-ci

Parsoid: Remove the TID suffix, if present

As per RfC7232, ETags can be suffixed depending on the content type sent.
Normally, the suffix should not be present in the If-Match header sent
my the client. Alas, in certain circumstances (yet to be exactly pinned
down) VE sends us an If-Match header that is suffixed by `-df`
(presumably indicating deflate-compressed content, which is not the
case), causing RESTBase to throw 404s for content that is in reality
available in storage.

This has been happening for 15 days now with at least 2000 occurrences
per day, which is problematic since users are unable to save their
modifications because of it. To work around this problem, RESTBase now
strips the suffix and complains only by logging the event.

Bug: T230272

2199 of 3048 branches covered (72.15%)

10 of 10 new or added lines in 2 files covered. (100.0%)

4203 of 4986 relevant lines covered (84.3%)

558.17 hits per line