Page MenuHomePhabricator

Mobile-Sections returns missing images
Closed, ResolvedPublic

Description

Not sure if this is a problem with the wiki or MCS itself...

When using this query: https://bm.wikipedia.org/api/rest_v1/page/mobile-sections/Seshel

The response HTML includes a reference to this image: //upload.wikimedia.org/wikipedia/commons/thumb/a/a1/Coat_of_arms_of_the_Seychelles.svg/125px-Coat_of_arms_of_the_Seychelles.svg.png

This leads to a 404, and causes MWOffliner to have missing images in scrapes:
https://github.com/openzim/mwoffliner/issues/616

Event Timeline

Isnit001 created this task.Mar 4 2019, 9:51 AM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMar 4 2019, 9:51 AM
Arlolra added a subscriber: Arlolra.Mar 4 2019, 7:50 PM

Hmm, so the file was renamed recently on 2018-02-10,
https://commons.wikimedia.org/w/index.php?title=File%3ACoat_of_arms_of_Seychelles.svg&type=revision&diff=338463956&oldid=338257332

Asking for a purge (?action=purge) rerendered the page correctly,
https://bm.wikipedia.org/api/rest_v1/page/html/Seshel

It looks like, because Parsoid links directly to thumbnailed images, we need changeprop to tell us to rerender all the pages linking to the file. Hopefully renames are rare.

ssastry closed this task as Resolved.Mon, Jun 10, 8:39 PM
ssastry claimed this task.
ssastry added subscribers: Pchelolo, mobrovac, ssastry.

There is nothing to do here on MCS or Parsoid end. @Pchelolo @mobrovac please reopean and reassign to changeprop OR create a new phab task for changeprop related to renames and rerendering.

ssastry triaged this task as Normal priority.Mon, Jun 10, 8:40 PM
ssastry reassigned this task from ssastry to Arlolra.