PHP Notice: Undefined variable: etag (from ResourceLoader.php)
Closed, ResolvedPublicPRODUCTION ERROR
Actions

Assigned To

Authored By

	Krinkle
	Jul 11 2022, 7:36 PM

Description

Error

mwversion: 1.39.0-wmf.19
reqId: 207fa137-112f-4c40-b461-9d5359131d6b
Find reqId in Logstash

normalized_message

[{reqId}] {exception_url}   PHP Notice: Undefined variable: etag

exception.trace

from /srv/mediawiki/php-1.39.0-wmf.19/includes/ResourceLoader/ResourceLoader.php(856)
#0 /srv/mediawiki/php-1.39.0-wmf.19/includes/ResourceLoader/ResourceLoader.php(856): MWExceptionHandler::handleError(integer, string, string, integer, array)
#1 /srv/mediawiki/php-1.39.0-wmf.19/load.php(53): MediaWiki\ResourceLoader\ResourceLoader->respond(MediaWiki\ResourceLoader\Context)
#2 /srv/mediawiki/php-1.39.0-wmf.19/load.php(39): wfLoadMain()
#3 /srv/mediawiki/w/load.php(3): require(string)
#4 {main}

Impact

Notes

Details

Request URL: https://en.wikipedia.org/w/load.php?lang=*&modules=*&only=*&skin=*&user=*&version=*

Related Changes in Gerrit:

	Subject	Repo	Branch	Lines +/-
	ResourceLoader: Remove redundant getCombinedVersion() try-catch	mediawiki/core	master	+1 -8
	ResourceLoader: Remove outer TimeoutException try-catch in respond()	mediawiki/core	master	+74 -79

Customize query in gerrit

Related Objects

Mentioned Here: rMWca71e69fc6fc: Try not to discard Excimer timeout exceptions

Event Timeline

Krinkle created this task.Jul 11 2022, 7:36 PM

Krinkle triaged this task as High priority.

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJul 11 2022, 7:36 PM

The logic in this file looks wonky indeed.

		$response = '';
		try {
			// ..
			$versionHash = '/* .. */';
			$etag = 'W/"' . $versionHash . '"';
			// ..
		} catch ( TimeoutException $e ) {
			$this->outputErrorAndLog( $e, 'Request timed out' );
		}
		$this->sendResponseHeaders( $context, $etag, (bool)$this->errors, $this->extraHeaders );

I'm not sure why we do an inline timeout handler like this. I'll try to dig up how we ended up with that. We'd probably want to localise that a bit further in some way, or just let it bubble up to the top layer as HTTP 5xx.

Krinkle moved this task from Inbox, needs triage to Doing: Prio Interrupt on the Performance-Team board.Jul 12 2022, 1:14 AM

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

[mediawiki/core@master] ResourceLoader: Remove outer TimeoutException try-catch in respond()

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

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

[mediawiki/core@master] ResourceLoader: Remove redundant getCombinedVersion() try-catch

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

The bug was introduced in February this year as part of https://gerrit.wikimedia.org/r/758592 (ca71e69fc6).

Krinkle moved this task from Inbox to Confirmed Problem on the MediaWiki-ResourceLoader board.Jul 29 2022, 11:02 PM

Krinkle closed this task as Resolved.Aug 3 2022, 12:55 AM

Change 817924 merged by jenkins-bot: