This repository was archived by the owner on Apr 26, 2024. It is now read-only.
-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
This repository was archived by the owner on Apr 26, 2024. It is now read-only.
add_threepid email confirmation emails link to /unstable/ namespace #9197
Copy link
Copy link
Open
Labels
A-3PID3rd party identifiers: e.g. email, phone number3rd party identifiers: e.g. email, phone numberO-OccasionalAffects or can be seen by some users regularly or most users rarelyAffects or can be seen by some users regularly or most users rarelyS-TolerableMinor significance, cosmetic issues, low or no impact to users.Minor significance, cosmetic issues, low or no impact to users.T-TaskRefactoring, removal, replacement, enabling or disabling functionality, other engineering tasks.Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks.Z-CleanupThings we want to get rid of, but aren't actively causing painThings we want to get rid of, but aren't actively causing pain
Description
When you add an email address to an account, you get an email which invites you to click on a link whose path is https://<server>/_matrix/client/unstable/add_threepid/email/submit_token?token=...&client_secret=....
We should not be using an unstable prefix for stable features. The correct path for this would be something under https://<server>/_synapse/client/.... Of course we'll have to support both for a while to avoid breaking the links in any emails that have already been sent.
(This was added in #6042, and probably should have been cleaned up as part of finalising matrix-org/matrix-spec-proposals#2290)
Metadata
Metadata
Assignees
Labels
A-3PID3rd party identifiers: e.g. email, phone number3rd party identifiers: e.g. email, phone numberO-OccasionalAffects or can be seen by some users regularly or most users rarelyAffects or can be seen by some users regularly or most users rarelyS-TolerableMinor significance, cosmetic issues, low or no impact to users.Minor significance, cosmetic issues, low or no impact to users.T-TaskRefactoring, removal, replacement, enabling or disabling functionality, other engineering tasks.Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks.Z-CleanupThings we want to get rid of, but aren't actively causing painThings we want to get rid of, but aren't actively causing pain