Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Remove unstable MSC2858 APIs #10103

Closed
7 of 9 tasks
callahad opened this issue Jun 1, 2021 · 12 comments · Fixed by #10693
Closed
7 of 9 tasks

Remove unstable MSC2858 APIs #10103

callahad opened this issue Jun 1, 2021 · 12 comments · Fixed by #10693
Assignees
Labels
T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks. Z-Future-Maintenance Things that can't yet be done, but will need cleaning up in a couple of months/releases

Comments

@callahad
Copy link
Contributor

callahad commented Jun 1, 2021

Followup to #9627 for APIs defined in matrix-org/matrix-spec-proposals#2858

We're aiming to implement this no sooner than the first release in August. By the calendar, that should be Synapse 1.38 1.40. Thus, we can implement any time after 1.37 1.39 has branched.

Blockers:

Tasks:

  • Notify client teams of impending change and get sign-off
    • Element iOS
    • Element Android
    • Element Web
  • Disable experimental flag on matrix.org
  • Monitor for a few days
  • Remove code from develop
@callahad callahad added T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks. Z-Future-Maintenance Things that can't yet be done, but will need cleaning up in a couple of months/releases labels Jun 1, 2021
@richvdh
Copy link
Member

richvdh commented Jul 29, 2021

The first release in August is going to be 1.40, for the record.

@richvdh
Copy link
Member

richvdh commented Jul 29, 2021

MSC2858 support is gated by an experimental flag, so the first step here is probably to turn that off on matrix.org.

@callahad
Copy link
Contributor Author

@richvdh notes that we can check if anyone is calling the _unstable endpoint.

@callahad
Copy link
Contributor Author

@richvdh to do that, and @reivilibre to observer (welcome (back) to the team!)

@reivilibre
Copy link
Contributor

reivilibre commented Jul 29, 2021

I went through the haproxy logs (thanks for your help Rich!) on matrix.org and found that we have roughly between 20 and 100 hits to this unstable API per hour.

I took a sample of one hour on one haproxy and found that all 47 requests were from iPhones.

There is already an issue to switch to the stable API, which I have commented on: element-hq/element-ios#4362 .

@reivilibre
Copy link
Contributor

reivilibre commented Jul 29, 2021

N.B. just for the sake of it, I found element-hq/element-android#3442 — Android has already changed over

& element-hq/element-web#17521 — web too

@richvdh
Copy link
Member

richvdh commented Aug 3, 2021

The element iOS team expect to have an update in the app store in two weeks, and recommend waiting another week after that, for 95% penetration of the new version.

So, we should be able to revisit this w/c 23rd August.

@richvdh
Copy link
Member

richvdh commented Aug 26, 2021

We're still seeing a few hits to the old API (like, about 6 per hour) from versions of Element-iOS older than 1.5.0. I spoke to @SBiOSoftWhare and while about 94% of people on element-ios are on the latest version, we'd like that figure to be a bit higher before we turn this off. Let's revisit in another week.

@squahtx
Copy link
Contributor

squahtx commented Sep 2, 2021

We checked with the Element iOS team again and they're seeing 94% adoption of 1.5.1 today, using a possibly more accurate metric(?). We'll give it another week.

@squahtx squahtx changed the title Remove unstable MSC2858 APIs [2021-09-09: Review] Remove unstable MSC2858 APIs Sep 2, 2021
@squahtx
Copy link
Contributor

squahtx commented Sep 9, 2021

Today (2021-09-09), 99.8% of iOS users are on 1.5.0+ according to the Element iOS team.

Between 2021-09-01 and 2021-09-08 we've also seen a drop in the rate of requests to the deprecated endpoints, from 37/hour to 10/hour.

@squahtx squahtx changed the title [2021-09-09: Review] Remove unstable MSC2858 APIs Remove unstable MSC2858 APIs Sep 9, 2021
@squahtx squahtx reopened this Sep 9, 2021
@squahtx
Copy link
Contributor

squahtx commented Sep 9, 2021

Now waiting for #10693 to be deployed on matrix.org

@squahtx
Copy link
Contributor

squahtx commented Sep 10, 2021

Live on matrix.org now.

@squahtx squahtx closed this as completed Sep 10, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks. Z-Future-Maintenance Things that can't yet be done, but will need cleaning up in a couple of months/releases
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants