ApiMobileView fails with ErrorException
Closed, ResolvedPublic2 Estimated Story PointsPRODUCTION ERROR
Actions

Assigned To

None

Authored By

	pmiazga
	May 29 2017, 10:03 PM

Description

From time to time ApiMobileViews logs an error which adds noise to our ability to see other errors:

/w/api.php ErrorException from line 131 of /srv/mediawiki/php-1.30.0-wmf.1/extensions/MobileFrontend/includes/api/ApiMobileView.php: PHP Error: Argument 1 passed to ApiResult::setArrayType() must be an instance of array, null given

Stack trace:

#0 /srv/mediawiki/php-1.30.0-wmf.1/includes/api/ApiResult.php(738): MWExceptionHandler::handleError(integer, string, string, integer, array, array)
#1 /srv/mediawiki/php-1.30.0-wmf.1/extensions/MobileFrontend/includes/api/ApiMobileView.php(131): ApiResult::setArrayType(NULL, string)
#2 /srv/mediawiki/php-1.30.0-wmf.1/includes/api/ApiMain.php(1578): ApiMobileView->execute()
#3 /srv/mediawiki/php-1.30.0-wmf.1/includes/api/ApiMain.php(545): ApiMain->executeAction()
#4 /srv/mediawiki/php-1.30.0-wmf.1/includes/api/ApiMain.php(516): ApiMain->executeActionWithErrorHandling()
#5 /srv/mediawiki/php-1.30.0-wmf.1/api.php(83): ApiMain->execute()
#6 /srv/mediawiki/w/api.php(3): include(string)
#7 {main}

First occurence: May 16th
Errors logged: 32

Logs: https://logstash.wikimedia.org/app/kibana#/dashboard/Reading-Web?_g=(refreshInterval:(display:Off,pause:!f,value:0),time:(from:now-30d,mode:quick,to:now))&_a=(filters:!(),options:(darkTheme:!f),panels:!((col:1,id:All-Events,panelIndex:1,row:1,size_x:6,size_y:2,type:visualization),(col:7,id:Top-20-Hosts,panelIndex:4,row:1,size_x:3,size_y:2,type:visualization),(col:10,id:MediaWiki-Versions,panelIndex:5,row:1,size_x:3,size_y:2,type:visualization),(col:1,id:'Trending-Messages-(Reduced-noice)',panelIndex:6,row:3,size_x:12,size_y:6,type:visualization),(col:1,columns:!(type,level,wiki,host,message),id:Default-Events-List,panelIndex:7,row:9,size_x:12,size_y:5,sort:!('@timestamp',desc),type:search)),query:(query_string:(analyze_wildcard:!t,query:'message:MobileFrontend%20%20AND%20exception.message:%20%22PHP%20Error:%20Argument%201%20passed%20to%20ApiResult::setArrayType()%20must%20be%20an%20instance%20of%20array,%20null%20given%22')),title:'Reading%20Web',uiState:(P-1:(spy:(mode:(fill:!f,name:!n))),P-4:(vis:(params:(sort:(columnIndex:!n,direction:!n)))),P-6:(vis:(params:(sort:(columnIndex:!n,direction:!n))))))

Line:

Details

	Subject	Repo	Branch	Lines +/-
	When page properties are not available do not throw exceptions	mediawiki/extensions/MobileFrontend	master	+87 -9

Customize query in gerrit

Related Objects

Mentioned In: T161026: PHP Warning: array_intersect_key() expects parameter 1 to be an array or collection
Mentioned Here: T161026: PHP Warning: array_intersect_key() expects parameter 1 to be an array or collection

Event Timeline

pmiazga created this task.May 29 2017, 10:03 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMay 29 2017, 10:03 PM

pmiazga added projects: MobileFrontend, Web-Team-Backlog.May 29 2017, 10:03 PM

pmiazga updated the task description. (Show Details)

Legoktm added a project: Wikimedia-production-error.May 29 2017, 11:43 PM

Jdlrobson triaged this task as High priority.May 30 2017, 4:04 PM

Jdlrobson moved this task from Incoming to Upcoming on the Web-Team-Backlog board.

Jdlrobson updated the task description. (Show Details)

Jdlrobson updated the task description. (Show Details)May 30 2017, 4:13 PM

Jdlrobson set the point value for this task to 2.May 30 2017, 4:15 PM

ovasileva added a project: Readers-Web-Kanbanana-Board-Old.May 31 2017, 12:03 PM

Jdlrobson moved this task from Upcoming to 2016-17 Q4 on the Web-Team-Backlog board.May 31 2017, 4:54 PM

• NHarateh_WMF moved this task from To Do to Needs Design Review on the Readers-Web-Kanbanana-Board-Old board.May 31 2017, 5:24 PM

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

Jdlrobson mentioned this in T161026: PHP Warning: array_intersect_key() expects parameter 1 to be an array or collection.May 31 2017, 7:39 PM

Change 356481 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/extensions/MobileFrontend@master] When page properties are not available do not throw exceptions

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

gerritbot added a project: Patch-For-Review.May 31 2017, 9:47 PM

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

Reviewed, looks ok

pmiazga moved this task from Needs Code Review to Ready for Signoff on the Readers-Web-Kanbanana-Board-Old board.May 31 2017, 10:49 PM

Change 356481 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@master] When page properties are not available do not throw exceptions

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

ReleaseTaggerBot added a project: MW-1.30-release-notes (WMF-deploy-2017-06-06_(1.30.0-wmf.4)).May 31 2017, 11:01 PM

• bmansurov claimed this task.Jun 1 2017, 5:04 PM

Will sign off once the patch is in production.

Will sign off tomorrow as the changes haven't hit production yet.

No error messages in the last 12 hours.

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

ApiMobileView fails with ErrorExceptionClosed, ResolvedPublic2 Estimated Story PointsPRODUCTION ERRORActions

Description

Details

Related Objects

Event Timeline

ApiMobileView fails with ErrorException
Closed, ResolvedPublic2 Estimated Story PointsPRODUCTION ERROR
Actions