MWExceptionRenderer::reportOutageHTML() produces invalid HTML
Open, Needs TriagePublic
Actions

Assigned To

None

Authored By

	labster
	Dec 24 2016, 5:22 AM

Description

Mainly because it doesn't produce an <html> tag. This error page is output when the database is down/unavailable. At least somewhat related to T149481, as there are also no lang or dir attributes either. Currently on REL1_28.

And while I'm here, can we maybe get an output hook in the outage HTML? I tried setting $wgExceptionHooks['DBConnectionError'], but did not get additional output. But this is an undocumented API, so maybe I'm doing it wrong?

Thanks!

Related Objects

Mentioned Here: T149481: MWExceptionRenderer doesn't declare the UI language and direction

Event Timeline

labster created this task.Dec 24 2016, 5:22 AM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptDec 24 2016, 5:22 AM

labster updated the task description. (Show Details)Dec 24 2016, 5:24 AM

Assuming this is about MediaWiki-General.

And while I'm here,

For separate requests, please file separate tasks. Thanks!

OK @Aklapper, I'll bite. Convince me that the second part is a separate task, and I'll open another task. I'm rather thinking that if the call to reportOutageHTML() moved into reportHTML() ... right around here ... that all of the above will be solved at the same time.

Just because the issue has come up again and I've done more reading since then... apparently <html> and <body> elements aren't required for valid HTML 5. But the <!DOCTYPE html> is missing, and that is required. Same suggested solution as in the comment above.

MWExceptionRenderer::reportOutageHTML() produces invalid HTMLOpen, Needs TriagePublicActions

Description

Related Objects

Event Timeline

MWExceptionRenderer::reportOutageHTML() produces invalid HTML
Open, Needs TriagePublic
Actions