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

Stabilise MSC3231 (Token Based Registration) #11867

Merged
merged 8 commits into from
Feb 4, 2022
Merged

Stabilise MSC3231 (Token Based Registration) #11867

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
cbb7579
stablize
ShadowJonathan Jan 31, 2022
f6bbd3f
add news
ShadowJonathan Jan 31, 2022
88834ec
whoops
ShadowJonathan Jan 31, 2022
5ee7386
another whoops
ShadowJonathan Jan 31, 2022
cc92c72
Update docs/modules/password_auth_provider_callbacks.md
ShadowJonathan Jan 31, 2022
54d841f
apply review feedback
ShadowJonathan Jan 31, 2022
d3ad0e8
apply review feedback
ShadowJonathan Feb 3, 2022
de2b881
Switch regular expressions to plain endpoint paths
anoadragon453 Feb 4, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 5 additions & 1 deletion changelog.d/11867.feature
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
Stabilize MSC3231.
Stabilize [MSC3231](https://github.com/matrix-org/matrix-doc/pull/3231).

Client implementations using `m.login.registration_token` should switch to the stable identifiers:
* `org.matrix.msc3231.login.registration_token` in query parameters and request/response bodies becomes `m.login.registration_token`.
* `/_matrix/client/unstable/org.matrix.msc3231/register/org.matrix.msc3231.login.registration_token/validity` becomes `/_matrix/client/v1/register/m.login.registration_token/validity`.
Comment on lines +1 to +5
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changelogs should only be on one line, I don't think this will be handled properly by towncrier.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I simplified this in 0408d69.

2 changes: 1 addition & 1 deletion docs/modules/password_auth_provider_callbacks.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -148,7 +148,7 @@ Here's an example featuring all currently supported keys:
"address": "33123456789",
"validated_at": 1642701357084,
},
"m.login.registration_token": "sometoken", # User has registered through registration tokens
"m.login.registration_token": "sometoken", # User has registered through a registration token
}
```

Expand Down
14 changes: 14 additions & 0 deletions docs/upgrade.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -84,6 +84,20 @@ process, for example:
wget https://packages.matrix.org/debian/pool/main/m/matrix-synapse-py3/matrix-synapse-py3_1.3.0+stretch1_amd64.deb
dpkg -i matrix-synapse-py3_1.3.0+stretch1_amd64.deb
```
# Upgrading to v1.(next)

## Stablisation of MSC3231

A reverse proxy rule needs to be updated;
ShadowJonathan marked this conversation as resolved.
Show resolved Hide resolved

`^/_matrix/client/unstable/org.matrix.msc3231/register/org.matrix.msc3231.login.registration_token/validity$`

To

`^/_matrix/client/v1/register/m.login.registration_token/validity$`

Any currently-active UIA session where a user is registering with a
registration token will be invalidated.
ShadowJonathan marked this conversation as resolved.
Show resolved Hide resolved

# Upgrading to v1.51.0

Expand Down