Watchstar hidden on unwatched pages - Watch icon from skins.minerva.icons.images.scripts invalid SVG ("PHP Warning: parsing XML declaration")
Closed, ResolvedPublicPRODUCTION ERROR
Actions

Description

Spotted in production. Two errors, but they come in pairs so they're clearly the same underlying bug:

Error reading SVG:Error domain 1 code 4 on line 1 column 1 of data: Start tag expected, '<' not found
Warning: parsing XML declaration: '?>' expected in Entity, line: 1 in /srv/mediawiki/php-1.30.0-wmf.16/includes/resourceloader/ResourceLoaderImage.php on line 302

ResourceLoader is making me think we've got a bad SVG that someone checked in somewhere? No stacktraces obvs cuz that would make life easy and shit.

=> Logstash: https://logstash.wikimedia.org/goto/d2698f1b9f332217fb797090b2bb5fa9

Replication steps

Visit https://m.mediawiki.org/wiki/Extension:MobileFrontend
If the watch star is green click it, if you can't see the watchstar at all that's the problem.
After clicking it, notice it's absence.

Details

	Subject	Repo	Branch	Lines +/-
	Fix watchstar icon	mediawiki/skins/MinervaNeue	wmf/1.30.0-wmf.16	+1 -1
	Fix watchstar icon	mediawiki/skins/MinervaNeue	master	+1 -1

Customize query in gerrit

Related Objects
Search...

		Status	Subtype	Assigned	Task
		Resolved	Release	• demon	T170634 1.30.0-wmf.16 deployment blockers
		Resolved	PRODUCTION ERROR	Jdlrobson	T174607 Watchstar hidden on unwatched pages - Watch icon from skins.minerva.icons.images.scripts invalid SVG ("PHP Warning: parsing XML declaration")

Event Timeline

• demon created this task.Aug 30 2017, 7:27 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptAug 30 2017, 7:27 PM

Setting to high because it's actually pretty spammy with wmf.16

• demon added subscribers: Krinkle, Catrope.Aug 30 2017, 7:28 PM

No stacktraces obvs cuz that would make life easy and shit.

Indeed. Except.. wait a minute... Lies! The error channel has traces now? (See update at T45086)

Logstash:

Screen Shot 2017-08-30 at 22.16.44.png (1×2 px, 363 KB)

#0 [internal function]: MWExceptionHandler::handleError(integer, string, string, integer, array, array)
#1 /srv/mediawiki/php-1.30.0-wmf.16/includes/resourceloader/ResourceLoaderImage.php(302): DOMDocument->loadXML(string)
#2 /srv/mediawiki/php-1.30.0-wmf.16/includes/resourceloader/ResourceLoaderImage.php(342): ResourceLoaderImage->massageSvgPathdata(string)
#3 /srv/mediawiki/php-1.30.0-wmf.16/includes/resourceloader/ResourceLoaderImage.php(243): ResourceLoaderImage->rasterize(string)
#4 /srv/mediawiki/php-1.30.0-wmf.16/includes/resourceloader/ResourceLoader.php(1050): ResourceLoaderImage->getImageData(ResourceLoaderContext)
#5 /srv/mediawiki/php-1.30.0-wmf.16/includes/resourceloader/ResourceLoader.php(783): ResourceLoader->makeModuleResponse(ResourceLoaderContext, array, array)
#6 /srv/mediawiki/php-1.30.0-wmf.16/load.php(53): ResourceLoader->respond(ResourceLoaderContext)
#7 /srv/mediawiki/w/load.php(3): include(string)
#8 {main}

Looking at https://logstash.wikimedia.org/goto/d2698f1b9f332217fb797090b2bb5fa9 it seems all the log entries are from urls requesting modules=skins.minerva.icons.images.scripts&image=watch.

Krinkle renamed this task from Bad SVG in ResourceLoader? to Watch icon from skins.minerva.icons.images.scripts invalid SVG ("PHP Warning: parsing XML declaration").Aug 30 2017, 9:52 PM

Krinkle updated the task description. (Show Details)

Krinkle edited projects, added MinervaNeue; removed MediaWiki-ResourceLoader.

Krinkle added a parent task: T170634: 1.30.0-wmf.16 deployment blockers.Aug 30 2017, 9:55 PM

The HVVM error channel (the first of the two errors) includes no stack trace. I assumed the other wouldn't either because too often the bugs I report don't have traces :(

Maybe introduced by https://gerrit.wikimedia.org/r/#/c/373960/1/resources/skins.minerva.icons.images.scripts/watch.svg,unified ?

ovasileva moved this task from Incoming to Upcoming on the Web-Team-Backlog board.Aug 31 2017, 11:25 AM

This is blocking the train and if it goes out on the train it will break the watchstar functionality, so it seems like a no-brainer that we should fix this today.

Jdlrobson updated the task description. (Show Details)Aug 31 2017, 1:39 PM

Jdlrobson moved this task from Upcoming to 2017-18 Q1 on the Web-Team-Backlog board.

Jdlrobson moved this task from Needs Design Review to Doing on the Readers-Web-Kanbanana-Board-Old board.Aug 31 2017, 5:11 PM

Jdlrobson moved this task from Doing to Needs Code Review on the Readers-Web-Kanbanana-Board-Old board.Aug 31 2017, 5:15 PM

Change 375028 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/skins/MinervaNeue@master] Fix watchstar icon

https://gerrit.wikimedia.org/r/375028

gerritbot added a project: Patch-For-Review.Aug 31 2017, 5:15 PM

Jdlrobson claimed this task.Aug 31 2017, 5:17 PM

Jdlrobson moved this task from Needs Code Review to Ready for Signoff on the Readers-Web-Kanbanana-Board-Old board.

@Jdlrobson to try to get this in before today's swat, else go with the old star

Change 375028 merged by jenkins-bot:
[mediawiki/skins/MinervaNeue@master] Fix watchstar icon

https://gerrit.wikimedia.org/r/375028

Change 375031 had a related patch set uploaded (by Chad; owner: Jdlrobson):
[mediawiki/skins/MinervaNeue@wmf/1.30.0-wmf.16] Fix watchstar icon

https://gerrit.wikimedia.org/r/375031

Change 375031 merged by jenkins-bot:
[mediawiki/skins/MinervaNeue@wmf/1.30.0-wmf.16] Fix watchstar icon

https://gerrit.wikimedia.org/r/375031

ReleaseTaggerBot added a project: MW-1.30-release-notes (WMF-deploy-2017-08-29 (1.30.0-wmf.16)).Aug 31 2017, 6:00 PM

Watchstar is now showing again on mediawiki.org

In T174607#3568793, @Jdlrobson wrote:

Maybe introduced by https://gerrit.wikimedia.org/r/#/c/373960/1/resources/skins.minerva.icons.images.scripts/watch.svg,unified ?

Initial thought, is the XML processing instruction invalid? It should end with an ?>, right? See https://en.wikipedia.org/wiki/Processing_Instruction

• mmodell changed the subtype of this task from "Task" to "Production Error".Aug 28 2019, 11:10 PM

	F9231044: Screen Shot 2017-08-30 at 22.16.44.png
	Aug 30 2017, 9:18 PM

Watchstar hidden on unwatched pages - Watch icon from skins.minerva.icons.images.scripts invalid SVG ("PHP Warning: parsing XML declaration")Closed, ResolvedPublicPRODUCTION ERRORActions