Page MenuHomePhabricator

MobileFrontend should not use raw HTML messages
Open, MediumPublic

Description

These seem to be the last remaining uses of raw HTML i18n messages in Wikimedia production:

  • mobile-frontend-editor-editing-page
  • mobile-frontend-editor-previewing-page
  • mobile-frontend-editor-summary
  • mobile-frontend-editor-summary-request
  • mobile-frontend-editor-wait

(codesearch)

Not exactly a security issue because they are listed in $wgRawHtmlMessages which makes them hard to edit, but we are trying to eliminate the attack vector entirely. At a glance it doesn't look like there's much reason for these to be raw HTML anyway.

Event Timeline

Jdlrobson-WMF moved this task from Incoming to Freezer on the Web-Team board.
Jdlrobson-WMF subscribed.

Need to review these messages.

mobile-frontend-editor-editing-page and mobile-frontend-editor-previewing-page could be parsed with jQueryMsg (mw.message(...).parse()), the other three don't appear to use any HTML/formatting. The only issue may be local wiki overwrites.

DLynch subscribed.

This doesn't seem to have anything to do with the VisualEditor on mobile. All these messages are related to the non-2017 source editor that MobileFrontend implemented.