Page MenuHomePhabricator
Create Task
Maniphest T170612

Only fetch content from new Page Summary MCS route when Accept header is v2.0+
Closed, DeclinedPublic
Actions

Description

NOTE: As the time of writing, v2.0 is the next major version of the Page Summary RESTBase endpoint.

We want to make sure that older clients still receive compatible responses (v1.2-) as the new Page Summary response contains a lot of breaking changes.

AC

  • For requests with the Accept: profile="https://www.mediawiki.org/wiki/Specs/Summary/1.2.0 HTTP header, the content is fetched from the MediaWiki API as it is now.
  • For requests with the Accept: profile="https://www.mediawiki.org/wiki/Specs/Summary/1.2.0 HTTP header, the content is fetched from the new Page Summary MCS route

Open Questions

  1. How many clients send requests without the profile part of the Accept HTTP header? See T170606: Add Accept header to webrequest logs
  2. At what point is returning v2.0+ content an acceptable default strategy?

Related Objects
Search...

Event Timeline

phuedx created this task.Jul 13 2017, 6:01 PM
Restricted Application added a project: Product-Infrastructure-Team-Backlog-Deprecated. · View Herald TranscriptJul 13 2017, 6:01 PM
Restricted Application added a subscriber: Aklapper. · View Herald Transcript
phuedx added a parent task: T168848: Bootstrap an initial version of the Page Summary API in MCS.Jul 13 2017, 6:02 PM
Jdlrobson added a project: Web-Team-Backlog-Archived (Tracking).Jul 13 2017, 6:05 PM
phuedx removed a parent task: T168848: Bootstrap an initial version of the Page Summary API in MCS.Jul 14 2017, 9:53 AM
phuedx added a subtask: T168848: Bootstrap an initial version of the Page Summary API in MCS.
• Fjalapeno moved this task from Needs triage to Tracking on the Product-Infrastructure-Team-Backlog-Deprecated board.Jul 14 2017, 4:58 PM
Jdlrobson moved this task from Untriaged to Move to Backlog on the Web-Team-Backlog-Archived (Tracking) board.Jul 17 2017, 9:48 PM
Jdlrobson edited projects, added Web-Team-Backlog-Archived; removed Web-Team-Backlog-Archived (Tracking).Jul 27 2017, 4:16 PM
Jdlrobson moved this task from Incoming to Upcoming on the Web-Team-Backlog-Archived board.
Jdlrobson subscribed.

When endpoint is ready we are likely to need to help this happen.

Jdlrobson changed the task status from Open to Stalled.Aug 2 2017, 6:33 PM

Blocked on T168848

Jdlrobson added a subscriber: • Pchelolo.Aug 31 2017, 7:03 PM

@Pchelolo is this still applicable (in relation to T168848?) or would it better to just switch outright?

• Pchelolo closed this task as Declined.Aug 31 2017, 9:35 PM

I think it's not applicable any more. Closing as declined.

Jdlrobson closed subtask T168848: Bootstrap an initial version of the Page Summary API in MCS as Resolved.Sep 7 2017, 4:15 PM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL · Credits