Page MenuHomePhabricator

Update php-wmerrors page to include request ID
Closed, ResolvedPublic

Description

Follow-up from T117128: The error page for bare exceptions sucks.

Screenshot 2021-09-16 at 15.22.09.png (1×1 px, 120 KB)

  • Mention the reqId on the page.
  • Consider adding a X-Request-Id response header as well, ref T201409.

Event Timeline

Change 721923 had a related patch set uploaded (by Krinkle; author: Krinkle):

[operations/puppet@production] mediawiki: Add request ID to php-wmerrors error page

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

Change 721924 had a related patch set uploaded (by Krinkle; author: Krinkle):

[operations/puppet@production] mediawiki: Set php-wmerrors reqId to \"unknown\" for Logstash

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

Krinkle triaged this task as Medium priority.Sep 18 2021, 4:12 AM

Change 721923 merged by Giuseppe Lavagetto:

[operations/puppet@production] mediawiki: Add request ID to php-wmerrors error page

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

Change 721924 merged by Giuseppe Lavagetto:

[operations/puppet@production] mediawiki: Set php-wmerrors reqId to \"unknown\" for Logstash

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