d00rman / restbase / 1001 / 1

84%

master: 84%

push

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

733 of 1016 branches covered (72.15%)

1401 of 1662 relevant lines covered (84.3%)

139.72 hits per line