Mholloway (Michael Holloway)
Software Engineer, WMF Readers Infrastructure

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

User Details

User Since
Mar 30 2015, 8:44 PM (146 w, 10 h)
Availability
Available
IRC Nick
mdholloway
LDAP User
Mholloway
MediaWiki User
MHolloway (WMF)

Recent Activity

Sun, Jan 14

Mholloway moved T69015: m.wikipedia.org and zero.wikipedia.org should redirect how/where from To Do to Code Review on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Sun, Jan 14, 3:45 PM · Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Zero, Discovery, Mobile, Wikimedia-Apache-configuration
Mholloway added a watcher for MediaWiki-Containers: Mholloway.
Sun, Jan 14, 3:22 AM

Fri, Jan 12

Mholloway added a comment to T184751: Benchmark the new page summary API.

What I'm thinking is that I'll spin up a medium or large Cloud VPS instance with local RESTBase and MCS installations and benchmark for a test sample before and after using ab and parallels along the lines described here: https://www.simonholywell.com/post/2015/06/parallel-benchmark-many-urls-with-apachebench/

Fri, Jan 12, 10:12 PM · Performance-Team, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)

Thu, Jan 11

Mholloway closed T184612: Uncaught Error: Call to undefined function wfDeprecated() when running populateWithTestData.php as Resolved.
Thu, Jan 11, 11:29 PM · MW-1.31-release-notes (WMF-deploy-2018-01-16 (1.31.0-wmf.17)), Patch-For-Review, Reading-Infrastructure-Team-Backlog, Reading List Service
Mholloway updated subscribers of T184751: Benchmark the new page summary API.
Thu, Jan 11, 6:10 PM · Performance-Team, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Mholloway placed T184753: Use cached page leads when creating page summaries to reduce MCS load up for grabs.
Thu, Jan 11, 6:09 PM · Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Mholloway created T184753: Use cached page leads when creating page summaries to reduce MCS load.
Thu, Jan 11, 6:09 PM · Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Mholloway updated subscribers of T184751: Benchmark the new page summary API.
Thu, Jan 11, 5:59 PM · Performance-Team, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Mholloway triaged T184751: Benchmark the new page summary API as High priority.
Thu, Jan 11, 5:58 PM · Performance-Team, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Mholloway added a project to T184669: Fetching incorrect description data from the Feed API: Multimedia.
Thu, Jan 11, 12:46 PM · Multimedia, Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Mobile-Content-Service
Mholloway renamed T184611: PHP Notice: Undefined index when running populateProjectsFromSiteMatrix.php and viewing Special:SiteMatrix in MW-Vagrant from Undefined index error when running populateProjectsFromSiteMatrix.php and viewing Special:SiteMatrix in MW-Vagrant to PHP Notice: Undefined index when running populateProjectsFromSiteMatrix.php and viewing Special:SiteMatrix in MW-Vagrant.
Thu, Jan 11, 3:54 AM · MediaWiki-Vagrant, SiteMatrix, Reading-Infrastructure-Team-Backlog, Reading List Service
Mholloway claimed T184669: Fetching incorrect description data from the Feed API.
Thu, Jan 11, 1:57 AM · Multimedia, Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Mobile-Content-Service
Mholloway moved T184669: Fetching incorrect description data from the Feed API from To Do to Code Review on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Thu, Jan 11, 1:57 AM · Multimedia, Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Mobile-Content-Service
Mholloway edited projects for T184669: Fetching incorrect description data from the Feed API, added: Reading-Infrastructure-Team-Backlog (Kanban); removed Multimedia, Reading-Infrastructure-Team-Backlog.
Thu, Jan 11, 1:56 AM · Multimedia, Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Mobile-Content-Service

Wed, Jan 10

Mholloway moved T175868: Deploy and test new book rendering (Remex + Electron) from Deployed to Blocked on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Wed, Jan 10, 3:56 PM · Electron-PDFs, MW-1.30-release-notes (WMF-deploy-2017-09-19 (1.30.0-wmf.19)), Readers-Web-Backlog, Proton, Reading-Infrastructure-Team-Backlog (Kanban)
Mholloway edited projects for T169897: Track print-related web standards, added: Reading-Infrastructure-Team-Backlog; removed Reading-Infrastructure-Team-Backlog (Kanban).
Wed, Jan 10, 3:55 PM · Reading-Infrastructure-Team-Backlog, Readers-Web-Backlog (Tracking)
Mholloway closed T177001: Run fundraising test via announcment cards in France as Resolved.
Wed, Jan 10, 3:54 PM · Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Wikipedia-Android-App-Backlog, Wikipedia-iOS-App-Backlog
Mholloway closed T158635: Editing section opens wikitext for previous section as Resolved.
Wed, Jan 10, 3:52 PM · Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, Mobile-Content-Service, Wikipedia-Android-App-Backlog, Android-app-Bugs
Mholloway added a subtask for T177431: Develop a Summary JSON API: Unknown Object (Task).
Wed, Jan 10, 3:51 PM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Mholloway closed T164291: Make title-related properties consistent as Resolved.
Wed, Jan 10, 3:50 PM · Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, RESTBase-API, Services (designing)
Mholloway closed T164291: Make title-related properties consistent, a subtask of T167022: Inventory requirements for summary 2.0 endpoint across Reading teams, as Resolved.
Wed, Jan 10, 3:50 PM · Services (watching), Reading-Infrastructure-Team-Backlog (Kanban)
Mholloway moved T171052: Add disambiguation page handling in Page Summary API from To Do to Blocked on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Wed, Jan 10, 3:46 PM · Reading-Infrastructure-Team-Backlog (Kanban), Readers-Web-Backlog (Tracking), Mobile-Content-Service, Page-Previews
Mholloway created T184612: Uncaught Error: Call to undefined function wfDeprecated() when running populateWithTestData.php.
Wed, Jan 10, 3:24 PM · MW-1.31-release-notes (WMF-deploy-2018-01-16 (1.31.0-wmf.17)), Patch-For-Review, Reading-Infrastructure-Team-Backlog, Reading List Service
Mholloway updated the task description for T184611: PHP Notice: Undefined index when running populateProjectsFromSiteMatrix.php and viewing Special:SiteMatrix in MW-Vagrant.
Wed, Jan 10, 3:19 PM · MediaWiki-Vagrant, SiteMatrix, Reading-Infrastructure-Team-Backlog, Reading List Service
Mholloway created T184611: PHP Notice: Undefined index when running populateProjectsFromSiteMatrix.php and viewing Special:SiteMatrix in MW-Vagrant.
Wed, Jan 10, 3:18 PM · MediaWiki-Vagrant, SiteMatrix, Reading-Infrastructure-Team-Backlog, Reading List Service

Tue, Jan 9

Mholloway added a comment to T173537: Phab file uploads are blocked for inactive Zero IP ranges.

Thanks, @Mholloway. It may be possible to check the HTTP headers on the response, but there too I'm unsure. In any event, I appreciate your testing.

Confirmed that the zeroconfig API returns an empty object for requests in configured but disabled IP ranges. However, responses in these cases will include an x-carrier header with the carrier's MCC-MNC code.

Tue, Jan 9, 8:40 PM · Patch-For-Review, Zero, Phabricator
Mholloway added a comment to T173537: Phab file uploads are blocked for inactive Zero IP ranges.

As far as ways for users to identify the tagging disposition, @Mholloway please let me know if this is or isn't your observation...but...they can go to the following URL:

https://en.wikipedia.org/w/api.php?action=zeroconfig&type=config

I'll test this again to be 100% sure, but unfortunately if memory serves (and as the source suggests) I believe an empty JSON object is also returned in the case of an existing but disabled carrier config. The fields are only populated if the config exists and is enabled.

Tue, Jan 9, 7:42 PM · Patch-For-Review, Zero, Phabricator
Mholloway added a comment to T183980: Phabricator shows no images but 403 error as the used internet provider is an active Zero partner.

@Zoranzoki21, using Orion, would you mind visiting https://en.wikipedia.org/w/api.php?action=zeroconfig&type=config and copy/pasting the result here?

Tue, Jan 9, 7:34 PM · Zero, Phabricator
Mholloway added a comment to T183980: Phabricator shows no images but 403 error as the used internet provider is an active Zero partner.

You're correct that Orion Telekom is not and appears never to have been a Zero partner.

Tue, Jan 9, 6:30 PM · Zero, Phabricator
Mholloway updated subscribers of T173537: Phab file uploads are blocked for inactive Zero IP ranges.

As for T173537#3886490, I believe that configured carrier IPs are consolidated for Varnish in https://github.com/wikimedia/mediawiki-extensions-ZeroPortal/blob/master/includes/ApiZeroPortal.php and that the logic there could be updated to exclude disabled carriers. (+@dr0ptp4kt for sanity checking).

Tue, Jan 9, 3:22 PM · Patch-For-Review, Zero, Phabricator
Mholloway added a comment to T183980: Phabricator shows no images but 403 error as the used internet provider is an active Zero partner.

Unfortunately, I believe this is expected behavior. Separately from the specific IPs in phabbanlist.conf, all users in Zero IP ranges are currently blocked from accessing uploaded files in Phabricator via https://gerrit.wikimedia.org/r/#/c/363264 (see T168142 for background on the abuse that led to this block).

Tue, Jan 9, 3:17 PM · Zero, Phabricator
Mholloway added a project to T183980: Phabricator shows no images but 403 error as the used internet provider is an active Zero partner: Zero.
Tue, Jan 9, 3:14 PM · Zero, Phabricator
Mholloway added a comment to T173537: Phab file uploads are blocked for inactive Zero IP ranges.

I think this issue may be affecting @Zoranzoki21 who reported this on irc.

Not only, also in T183980. (Which might turn out to be a duplicate?)

Telenor Serbia is an active Zero partner; unfortunately for that user, the patch is working as intended in that case.

Tue, Jan 9, 3:14 PM · Patch-For-Review, Zero, Phabricator
Mholloway renamed T173537: Phab file uploads are blocked for inactive Zero IP ranges from Can't view images on Phabricator but not on WP0 to Phab file uploads are blocked for inactive Zero IP ranges.
Tue, Jan 9, 2:31 PM · Patch-For-Review, Zero, Phabricator
Mholloway added a comment to T173537: Phab file uploads are blocked for inactive Zero IP ranges.

I was able to reproduce this, and I think @wassan.anmol117 is correct that this is caused by https://gerrit.wikimedia.org/r/#/c/363264/. Varnish internally marks up requests in Zero IP ranges with X-CS headers. The patch blocks requests marked up with this header from requesting uploaded files from Phabricator. The problem (if I'm correct) is that the X-CS markup is applied without regard to whether or not a carrier is a currently active Zero participant. (It's up to clients to determine whether the carrier referred to in a Zero header is active before displaying Zero chrome.)

Tue, Jan 9, 2:29 PM · Patch-For-Review, Zero, Phabricator
Mholloway added a comment to T173537: Phab file uploads are blocked for inactive Zero IP ranges.

Something's definitely awry. Orange Cameroon has not been an active Zero partner since March 2016. I'm looking into it.

Tue, Jan 9, 1:30 PM · Patch-For-Review, Zero, Phabricator
Mholloway added a comment to T164739: Allow page previews to display in wiktionary.

See also: T176242: [EPIC] Representing / extracting wiki-specific application-level semantics

Tue, Jan 9, 12:55 PM · Need-volunteer, Reading-Infrastructure-Team-Backlog, Mobile-Content-Service, Wiktionary, Readers-Web-Backlog (Tracking), Epic, Page-Previews
Mholloway added a comment to T164739: Allow page previews to display in wiktionary.

Also, there's no concept of a primary definition in any Wiktionary as far as I'm aware, so that would have to be defined in a product spec.

Tue, Jan 9, 12:51 PM · Need-volunteer, Reading-Infrastructure-Team-Backlog, Mobile-Content-Service, Wiktionary, Readers-Web-Backlog (Tracking), Epic, Page-Previews
Mholloway updated subscribers of T164739: Allow page previews to display in wiktionary.
Tue, Jan 9, 12:49 PM · Need-volunteer, Reading-Infrastructure-Team-Backlog, Mobile-Content-Service, Wiktionary, Readers-Web-Backlog (Tracking), Epic, Page-Previews
Mholloway added a comment to T164739: Allow page previews to display in wiktionary.

I wouldn't hold the existing Wiktionary definition endpoint up as an example to emulate. IMO the only sane way of moving forward on Wiktionary integration, given the lack of consistency in page structure or content, is to come up with a set of markup reflecting elements we're interested in (definitions, examples, etc.), and then work with Wiktionary communities to incorporate that markup into pages so that clients can reliably query for what they're interested in. There's a task for this: T138709

Tue, Jan 9, 12:48 PM · Need-volunteer, Reading-Infrastructure-Team-Backlog, Mobile-Content-Service, Wiktionary, Readers-Web-Backlog (Tracking), Epic, Page-Previews

Mon, Jan 8

Mholloway closed T184430: Flag project main pages with type=mainpage, a subtask of T177431: Develop a Summary JSON API, as Resolved.
Mon, Jan 8, 11:30 PM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Mholloway closed T184430: Flag project main pages with type=mainpage as Resolved.
Mon, Jan 8, 11:30 PM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Mholloway closed T184429: Ensure "timestamp" refers to the requested revision, a subtask of T177431: Develop a Summary JSON API, as Resolved.
Mon, Jan 8, 11:30 PM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Mholloway closed T184429: Ensure "timestamp" refers to the requested revision as Resolved.
Mon, Jan 8, 11:30 PM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Mholloway moved T184430: Flag project main pages with type=mainpage from To Deploy to Deployed on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Mon, Jan 8, 11:30 PM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Mholloway moved T184429: Ensure "timestamp" refers to the requested revision from To Deploy to Deployed on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Mon, Jan 8, 11:30 PM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Mholloway renamed T184476: Stop bundling ext.cite.style in the Android app when Parsoid HTML wraps refs in <sup>? from Stop bundling ext.cite.style in the Android app when Parsoid HTML wraps refs in <sup> to Stop bundling ext.cite.style in the Android app when Parsoid HTML wraps refs in <sup>?.
Mon, Jan 8, 11:24 PM · Wikipedia-Android-App-Backlog (Android-app-release-v2.7.23x-G), Mobile-Content-Service, Reading-Infrastructure-Team-Backlog
Mholloway updated the task description for T184476: Stop bundling ext.cite.style in the Android app when Parsoid HTML wraps refs in <sup>?.
Mon, Jan 8, 11:14 PM · Wikipedia-Android-App-Backlog (Android-app-release-v2.7.23x-G), Mobile-Content-Service, Reading-Infrastructure-Team-Backlog
Mholloway moved T184476: Stop bundling ext.cite.style in the Android app when Parsoid HTML wraps refs in <sup>? from Needs triage to Tracking on the Reading-Infrastructure-Team-Backlog board.
Mon, Jan 8, 8:59 PM · Wikipedia-Android-App-Backlog (Android-app-release-v2.7.23x-G), Mobile-Content-Service, Reading-Infrastructure-Team-Backlog
Mholloway updated the task description for T184476: Stop bundling ext.cite.style in the Android app when Parsoid HTML wraps refs in <sup>?.
Mon, Jan 8, 8:56 PM · Wikipedia-Android-App-Backlog (Android-app-release-v2.7.23x-G), Mobile-Content-Service, Reading-Infrastructure-Team-Backlog
Mholloway created T184476: Stop bundling ext.cite.style in the Android app when Parsoid HTML wraps refs in <sup>?.
Mon, Jan 8, 8:55 PM · Wikipedia-Android-App-Backlog (Android-app-release-v2.7.23x-G), Mobile-Content-Service, Reading-Infrastructure-Team-Backlog
Mholloway moved T184430: Flag project main pages with type=mainpage from Code Review to To Deploy on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Mon, Jan 8, 4:00 PM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Mholloway moved T184429: Ensure "timestamp" refers to the requested revision from Code Review to To Deploy on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Mon, Jan 8, 3:45 PM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Mholloway moved T184429: Ensure "timestamp" refers to the requested revision from Doing to Code Review on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Mon, Jan 8, 3:10 PM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Mholloway moved T184429: Ensure "timestamp" refers to the requested revision from To Do to Doing on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Mon, Jan 8, 2:39 PM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Mholloway moved T184430: Flag project main pages with type=mainpage from To Do to Code Review on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Mon, Jan 8, 2:39 PM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Mholloway created T184430: Flag project main pages with type=mainpage.
Mon, Jan 8, 1:35 PM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Mholloway created T184429: Ensure "timestamp" refers to the requested revision.
Mon, Jan 8, 1:34 PM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)

Thu, Jan 4

Mholloway closed T182330: Media: handle galleries as Resolved.
Thu, Jan 4, 6:59 PM · Reading-Infrastructure-Team-Backlog (Kanban), Page Content Service, Reading Epics (Platform JS CSS and HTML consolidation)
Mholloway closed T182330: Media: handle galleries, a subtask of T177430: Develop a Media JSON API, as Resolved.
Thu, Jan 4, 6:59 PM · Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Page Content Service, Reading Epics (Platform JS CSS and HTML consolidation)

Thu, Dec 21

Mholloway edited projects for T183347: mobile view rest api showing old content for current revision, added: RESTBase, ChangeProp, Services; removed MobileApp.
Thu, Dec 21, 3:46 AM · Services (doing), ChangeProp, RESTBase, Android-app-Bugs, Reading-Infrastructure-Team-Backlog, Wikipedia-Android-App-Backlog, Mobile-Content-Service

Dec 14 2017

Mholloway closed T182774: Improve section validation as Resolved.
Dec 14 2017, 8:44 PM · Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, Mobile-Content-Service
Mholloway moved T182868: On This Day endpoint returning empty list on frwiki from Code Review to To Deploy on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Dec 14 2017, 3:16 PM · Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Page Content Service, Mobile-Content-Service
Mholloway moved T182774: Improve section validation from Code Review to To Deploy on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Dec 14 2017, 3:16 PM · Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, Mobile-Content-Service

Dec 13 2017

Mholloway claimed T177431: Develop a Summary JSON API.
Dec 13 2017, 11:38 PM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Mholloway moved T177431: Develop a Summary JSON API from Blocked to Deployed on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Dec 13 2017, 11:38 PM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Mholloway changed the status of T179875: Update RESTBase to get summary content from MCS Summary 2.0 endpoint when development is complete, a subtask of T182321: Show HTML summaries on cswiki, from Open to Stalled.
Dec 13 2017, 11:37 PM · Readers-Web-Kanban-Board, Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Wikimedia-Site-requests, Page-Previews, Readers-Web-Backlog
Mholloway changed the status of T179875: Update RESTBase to get summary content from MCS Summary 2.0 endpoint when development is complete from Open to Stalled.
Dec 13 2017, 11:37 PM · Patch-For-Review, User-mobrovac, Services (doing), RESTBase-API, RESTBase, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Mholloway changed the status of T179875: Update RESTBase to get summary content from MCS Summary 2.0 endpoint when development is complete, a subtask of T177431: Develop a Summary JSON API, from Open to Stalled.
Dec 13 2017, 11:37 PM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Mholloway moved T179875: Update RESTBase to get summary content from MCS Summary 2.0 endpoint when development is complete from To Do to Blocked on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Dec 13 2017, 11:37 PM · Patch-For-Review, User-mobrovac, Services (doing), RESTBase-API, RESTBase, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Mholloway edited projects for T179876: Expose media endpoint in RESTBase when ready, added: Reading-Infrastructure-Team-Backlog; removed Reading-Infrastructure-Team-Backlog (Kanban).
Dec 13 2017, 11:36 PM · Reading-Infrastructure-Team-Backlog, Services (later), RESTBase-API, Page Content Service, Reading Epics (Platform JS CSS and HTML consolidation)
Mholloway moved T182774: Improve section validation from To Do to Code Review on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Dec 13 2017, 11:34 PM · Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, Mobile-Content-Service
Mholloway moved T69015: m.wikipedia.org and zero.wikipedia.org should redirect how/where from Blocked to To Do on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Dec 13 2017, 10:32 PM · Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Zero, Discovery, Mobile, Wikimedia-Apache-configuration
Mholloway closed T181743: iOS app users in California seeing both US and Canada banners as Resolved.
Dec 13 2017, 10:25 PM · Mobile-Content-Service, Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, iOS-app-v5.7.2, Wikipedia-iOS-App-Backlog, Wikimedia-Fundraising
Mholloway closed T181743: iOS app users in California seeing both US and Canada banners, a subtask of T181004: Run Big English fundraising on apps, as Resolved.
Dec 13 2017, 10:25 PM · Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Wikipedia-iOS-App-Backlog, Wikipedia-Android-App-Backlog, Wikimedia-Fundraising, Mobile-Content-Service
Mholloway moved T182774: Improve section validation from To Deploy to To Do on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Dec 13 2017, 10:24 PM · Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, Mobile-Content-Service
Mholloway closed T182802: Update announcements to stop English fundraiser on December 20, 2017 as Resolved.
Dec 13 2017, 10:24 PM · Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban)
Mholloway moved T182349: Section parsing bug on :en:Wikimedia Foundation from Needs triage to Tracking on the Reading-Infrastructure-Team-Backlog board.
Dec 13 2017, 9:11 PM · Patch-For-Review, Reading-Infrastructure-Team-Backlog, Parsoid, Mobile-Content-Service
Mholloway edited projects for T182349: Section parsing bug on :en:Wikimedia Foundation, added: Reading-Infrastructure-Team-Backlog; removed Reading-Infrastructure-Team-Backlog (Kanban).
Dec 13 2017, 9:11 PM · Patch-For-Review, Reading-Infrastructure-Team-Backlog, Parsoid, Mobile-Content-Service
Mholloway closed T182329: Media: handle items that link to documents other than a File page as Declined.

Per discussion in weekly RI meeting, no need for special handling of these since (unlike MediaViewer) we're not altering link behavior.

Dec 13 2017, 8:22 PM · Reading-Infrastructure-Team-Backlog (Kanban), Page Content Service, Reading Epics (Platform JS CSS and HTML consolidation)
Mholloway closed T182329: Media: handle items that link to documents other than a File page, a subtask of T177430: Develop a Media JSON API, as Declined.
Dec 13 2017, 8:22 PM · Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Page Content Service, Reading Epics (Platform JS CSS and HTML consolidation)
Mholloway moved T182330: Media: handle galleries from To Do to Code Review on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Dec 13 2017, 8:21 PM · Reading-Infrastructure-Team-Backlog (Kanban), Page Content Service, Reading Epics (Platform JS CSS and HTML consolidation)
Mholloway claimed T182774: Improve section validation.
Dec 13 2017, 8:20 PM · Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, Mobile-Content-Service
Mholloway moved T182774: Improve section validation from To Do to To Deploy on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Dec 13 2017, 8:20 PM · Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, Mobile-Content-Service
Mholloway edited projects for T182774: Improve section validation, added: Reading-Infrastructure-Team-Backlog (Kanban); removed Reading-Infrastructure-Team-Backlog.
Dec 13 2017, 8:20 PM · Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, Mobile-Content-Service
Mholloway added a comment to T182774: Improve section validation.

@ssastry, https://fr.wikipedia.org/api/rest_v1/page/html/Mod%C3%A8le:Donn%C3%A9es%2F6117%2F%C3%A9volution_population is an illustrative example of a page that caused an error yesterday. Of interest to your comment, the first section identified contains only a <p> and a <span> as children, no headings. The second section contains headings but only in child sections.

Dec 13 2017, 7:05 PM · Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, Mobile-Content-Service
Mholloway updated the task description for T181004: Run Big English fundraising on apps.
Dec 13 2017, 6:11 PM · Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Wikipedia-iOS-App-Backlog, Wikipedia-Android-App-Backlog, Wikimedia-Fundraising, Mobile-Content-Service
Mholloway renamed T182774: Improve section validation from Figure out how to handle non-content "sections" to Improve section validation.
Dec 13 2017, 4:00 PM · Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, Mobile-Content-Service
Mholloway updated the task description for T182774: Improve section validation.
Dec 13 2017, 3:59 PM · Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, Mobile-Content-Service
Mholloway added a comment to T182774: Improve section validation.

I'm also skeptical that throwing an error is the right way of going about things in the situations we do care about.

Dec 13 2017, 3:57 PM · Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, Mobile-Content-Service
Mholloway added a comment to T182774: Improve section validation.

A quick and probably mostly effective fix would be to exclude sections with IDs of -1 and -2 from this checking. I'm trying to think through whether this is correct behavior in light of what those IDs represent.

Dec 13 2017, 3:51 PM · Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, Mobile-Content-Service
Mholloway updated the task description for T182774: Improve section validation.
Dec 13 2017, 3:25 PM · Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, Mobile-Content-Service
Mholloway added a comment to T182770: meta property="dc:modified" may be absent.

T182728: Mismatched HTML + data-parsoid being processed by Parsoid on VE saves causing page corruptions might be related.

Dec 13 2017, 3:22 PM · Services (done), RESTBase, Reading-Infrastructure-Team-Backlog, Mobile-Content-Service
Mholloway updated the task description for T182770: meta property="dc:modified" may be absent.
Dec 13 2017, 3:09 PM · Services (done), RESTBase, Reading-Infrastructure-Team-Backlog, Mobile-Content-Service
Mholloway updated the task description for T182770: meta property="dc:modified" may be absent.
Dec 13 2017, 3:08 PM · Services (done), RESTBase, Reading-Infrastructure-Team-Backlog, Mobile-Content-Service
Mholloway triaged T182774: Improve section validation as High priority.
Dec 13 2017, 2:51 PM · Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, Mobile-Content-Service
Mholloway created T182774: Improve section validation.
Dec 13 2017, 2:51 PM · Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, Mobile-Content-Service
Mholloway added a comment to T182770: meta property="dc:modified" may be absent.

Actually, there seems to be a bigger problem here. /page/html is returning only sections of the desired page.

Dec 13 2017, 2:38 PM · Services (done), RESTBase, Reading-Infrastructure-Team-Backlog, Mobile-Content-Service
Mholloway updated the task description for T182770: meta property="dc:modified" may be absent.
Dec 13 2017, 2:24 PM · Services (done), RESTBase, Reading-Infrastructure-Team-Backlog, Mobile-Content-Service
Mholloway updated the task description for T182770: meta property="dc:modified" may be absent.
Dec 13 2017, 2:23 PM · Services (done), RESTBase, Reading-Infrastructure-Team-Backlog, Mobile-Content-Service