Replace trust_identity_server_for_password_resets with account_threepid_delegate #5876

anoadragon453 · 2019-08-19T13:16:02Z

Based on ~~#5875~~

Replaces the trust_identity_server_for_password_resets boolean option with a account_threepid_delegate str option that defines which identity server to use to handle password resets and registration if the homeserver does not want to or is unable to handle these tasks itself.

Having this option being something other than null or "" gives the same indication as True for trust_identity_server_for_password_resets.

The domain of the identity server is actually used in #5835

…id_delegate

docs/sample_config.yaml

synapse/config/emailconfig.py

erikjohnston · 2019-08-19T14:15:11Z

synapse/config/emailconfig.py

-        if self.email_password_reset_behaviour == "local" and email_config == {}:
+        # Prior to Synapse v1.4.0, there used to be another option that defined whether Synapse
+        # would use an identity server to password reset tokens on its behalf. We now warn the
+        # user if they have this set and tell them to use the updated option.


This doesn't warn but kills synapse. I'd strongly prefer to avoid doing this, as otherwise we need to scream from the roof tops that the upgrade may cause your synapse to not start. Can we do something backwards compatible?

We need to know what identity server they'd prefer to use to handle registration and password resets. We could just warn them with a message on startup without blocking Synapse start? Not exactly sure where in the codebase we could put the message though, given we can't log here.

Can we keep the old behaviour? Of using the first entry in trusted_id_servers, or whatever it does now?

We can, though that's another thing relying on the trusted id servers list.

Though this should also be eventually removed, so I say that could probably work.

Though if their trusted_third_party_id_servers is empty and trust_identity_server_for_password_resets is set we'll just ConfigError?

Though if their trusted_third_party_id_servers is empty and trust_identity_server_for_password_resets is set we'll just ConfigError?

This is what I've done.

synapse/config/emailconfig.py

synapse/handlers/identity.py

Co-Authored-By: Erik Johnston <erik@matrix.org>

…:matrix-org/synapse into anoa/reg_email_account_threepid_delegate * 'anoa/reg_email_account_threepid_delegate' of github.com:matrix-org/synapse: Update synapse/handlers/identity.py Update synapse/handlers/identity.py Update synapse/config/emailconfig.py

…legate * anoa/reg_email: Remove trusted_third_party_id_servers functionality (#5875)

synapse/config/emailconfig.py

erikjohnston · 2019-08-20T09:37:41Z

synapse/handlers/auth.py

-            not password_servlet
-            or self.hs.config.email_password_reset_behaviour == "remote"
-        ):
+        if not password_servlet or self.hs.config.email_threepid_behaviour == "remote":


Is doing this for non password reset still the correct thing to do?

This is going to be removed in part 2 of #5835

synapse/handlers/identity.py

synapse/config/emailconfig.py

Co-Authored-By: Erik Johnston <erik@matrix.org>

anoadragon453 · 2019-08-20T13:39:19Z

synapse/rest/client/v2_alpha/register.py

            raise SynapseError(400, "Email is already in use", Codes.THREEPID_IN_USE)

-        ret = yield self.identity_handler.requestEmailToken(**body)
+        ret = yield self.identity_handler.requestEmailToken(
+            None, email, client_secret, send_attempt, next_link


/register isn't going to allow the client to specify the id_server any more, so we don't pass it on.

synapse/handlers/identity.py

Co-Authored-By: Erik Johnston <erik@matrix.org>

synapse/rest/client/v2_alpha/account.py

…il (#5835) ~~Fixes #5833 Moved out to ~~#5863 Part of fixing #5751 Decouples the activity of sending registration emails and binding them to an identity server. This PR simply sends the registration email, but clicking it does not approve the user for registration. That will come in PR #2. Some of this makes use of existing stuff for sending password reset emails from Synapse. Some work was done to make that stuff even more generic. Upgrade notes: * There is a new top-level config option, `account_threepid_delegate` which defines the address of an identity server that you would like to send registration/password reset emails on your behalf. The option `email.trust_identity_server_for_password_resets` has been replaced by this. If you set `email.trust_identity_server_for_password_resets` in your config to `true`, please remove it and configure `account_threepid_delegate` instead. The [sample config](https://github.com/matrix-org/synapse/blob/anoa/reg_email_sending_email/docs/sample_config.yaml) has information on how to configure it. Note: This PR does not allow homeservers to send emails when simply adding an email to your account. That will come after this and will be blocked on a new MSC. Part [2/2] will be successfully completing the registration step when `/submit_token` is hit with the correct details, and clearing out the `password_servlet` flag stuff, which is no longer needed. Will be a much smaller PR than this one. ~~Requires #5863 has been merged into the base branch. ~~Requires #5876 has been merged into the base branch.

…rnal server to handle 3pid validation (#5987) This is a combination of a few different PRs, finally all being merged into `develop`: * #5875 * #5876 * #5868 (This one added the `/versions` flag but the flag itself was actually [backed out](891afb5#diff-e591d42d30690ffb79f63bb726200891) in #5969. What's left is just giving /versions access to the config file, which could be useful in the future) * #5835 * #5969 * #5940 Clients should not actually use the new registration functionality until #5972 is merged. UPGRADE.rst, changelog entries and config file changes should all be reviewed closely before this PR is merged.

@Sorunome

Synapse 1.4.0 (2019-10-03) ========================== Bugfixes -------- - Redact `client_secret` in server logs. ([\#6158](#6158)) Synapse 1.4.0rc2 (2019-10-02) ============================= Bugfixes -------- - Fix bug in background update that adds last seen information to the `devices` table, and improve its performance on Postgres. ([\#6135](#6135)) - Fix bad performance of censoring redactions background task. ([\#6141](#6141)) - Fix fetching censored redactions from DB, which caused APIs like initial sync to fail if it tried to include the censored redaction. ([\#6145](#6145)) - Fix exceptions when storing large retry intervals for down remote servers. ([\#6146](#6146)) Internal Changes ---------------- - Fix up sample config entry for `redaction_retention_period` option. ([\#6117](#6117)) Synapse 1.4.0rc1 (2019-09-26) ============================= Note that this release includes significant changes around 3pid verification. Administrators are reminded to review the [upgrade notes](UPGRADE.rst#upgrading-to-v140). Features -------- - Changes to 3pid verification: - Add the ability to send registration emails from the homeserver rather than delegating to an identity server. ([\#5835](#5835), [\#5940](#5940), [\#5993](#5993), [\#5994](#5994), [\#5868](#5868)) - Replace `trust_identity_server_for_password_resets` config option with `account_threepid_delegates`, and make the `id_server` parameteter optional on `*/requestToken` endpoints, as per [MSC2263](matrix-org/matrix-spec-proposals#2263). ([\#5876](#5876), [\#5969](#5969), [\#6028](#6028)) - Switch to using the v2 Identity Service `/lookup` API where available, with fallback to v1. (Implements [MSC2134](matrix-org/matrix-spec-proposals#2134) plus `id_access_token authentication` for v2 Identity Service APIs from [MSC2140](matrix-org/matrix-spec-proposals#2140)). ([\#5897](#5897)) - Remove `bind_email` and `bind_msisdn` parameters from `/register` ala [MSC2140](matrix-org/matrix-spec-proposals#2140). ([\#5964](#5964)) - Add `m.id_access_token` to `unstable_features` in `/versions` as per [MSC2264](matrix-org/matrix-spec-proposals#2264). ([\#5974](#5974)) - Use the v2 Identity Service API for 3PID invites. ([\#5979](#5979)) - Add `POST /_matrix/client/unstable/account/3pid/unbind` endpoint from [MSC2140](matrix-org/matrix-spec-proposals#2140) for unbinding a 3PID from an identity server without removing it from the homeserver user account. ([\#5980](#5980), [\#6062](#6062)) - Use `account_threepid_delegate.email` and `account_threepid_delegate.msisdn` for validating threepid sessions. ([\#6011](#6011)) - Allow homeserver to handle or delegate email validation when adding an email to a user's account. ([\#6042](#6042)) - Implement new Client Server API endpoints `/account/3pid/add` and `/account/3pid/bind` as per [MSC2290](matrix-org/matrix-spec-proposals#2290). ([\#6043](#6043)) - Add an unstable feature flag for separate add/bind 3pid APIs. ([\#6044](#6044)) - Remove `bind` parameter from Client Server POST `/account` endpoint as per [MSC2290](matrix-org/matrix-spec-proposals#2290). ([\#6067](#6067)) - Add `POST /add_threepid/msisdn/submit_token` endpoint for proxying submitToken on an `account_threepid_handler`. ([\#6078](#6078)) - Add `submit_url` response parameter to `*/msisdn/requestToken` endpoints. ([\#6079](#6079)) - Add `m.require_identity_server` flag to /version's unstable_features. ([\#5972](#5972)) - Enhancements to OpenTracing support: - Make OpenTracing work in worker mode. ([\#5771](#5771)) - Pass OpenTracing contexts between servers when transmitting EDUs. ([\#5852](#5852)) - OpenTracing for device list updates. ([\#5853](#5853)) - Add a tag recording a request's authenticated entity and corresponding servlet in OpenTracing. ([\#5856](#5856)) - Add minimum OpenTracing for client servlets. ([\#5983](#5983)) - Check at setup that OpenTracing is installed if it's enabled in the config. ([\#5985](#5985)) - Trace replication send times. ([\#5986](#5986)) - Include missing OpenTracing contexts in outbout replication requests. ([\#5982](#5982)) - Fix sending of EDUs when OpenTracing is enabled with an empty whitelist. ([\#5984](#5984)) - Fix invalid references to None while OpenTracing if the log context slips. ([\#5988](#5988), [\#5991](#5991)) - OpenTracing for room and e2e keys. ([\#5855](#5855)) - Add OpenTracing span over HTTP push processing. ([\#6003](#6003)) - Add an admin API to purge old rooms from the database. ([\#5845](#5845)) - Retry well-known lookups if we have recently seen a valid well-known record for the server. ([\#5850](#5850)) - Add support for filtered room-directory search requests over federation ([MSC2197](matrix-org/matrix-spec-proposals#2197), in order to allow upcoming room directory query performance improvements. ([\#5859](#5859)) - Correctly retry all hosts returned from SRV when we fail to connect. ([\#5864](#5864)) - Add admin API endpoint for setting whether or not a user is a server administrator. ([\#5878](#5878)) - Enable cleaning up extremities with dummy events by default to prevent undue build up of forward extremities. ([\#5884](#5884)) - Add config option to sign remote key query responses with a separate key. ([\#5895](#5895)) - Add support for config templating. ([\#5900](#5900)) - Users with the type of "support" or "bot" are no longer required to consent. ([\#5902](#5902)) - Let synctl accept a directory of config files. ([\#5904](#5904)) - Increase max display name size to 256. ([\#5906](#5906)) - Add admin API endpoint for getting whether or not a user is a server administrator. ([\#5914](#5914)) - Redact events in the database that have been redacted for a week. ([\#5934](#5934)) - New prometheus metrics: - `synapse_federation_known_servers`: represents the total number of servers your server knows about (i.e. is in rooms with), including itself. Enable by setting `metrics_flags.known_servers` to True in the configuration.([\#5981](#5981)) - `synapse_build_info`: exposes the Python version, OS version, and Synapse version of the running server. ([\#6005](#6005)) - Give appropriate exit codes when synctl fails. ([\#5992](#5992)) - Apply the federation blacklist to requests to identity servers. ([\#6000](#6000)) - Add `report_stats_endpoint` option to configure where stats are reported to, if enabled. Contributed by @Sorunome. ([\#6012](#6012)) - Add config option to increase ratelimits for room admins redacting messages. ([\#6015](#6015)) - Stop sending federation transactions to servers which have been down for a long time. ([\#6026](#6026)) - Make the process for mapping SAML2 users to matrix IDs more flexible. ([\#6037](#6037)) - Return a clearer error message when a timeout occurs when attempting to contact an identity server. ([\#6073](#6073)) - Prevent password reset's submit_token endpoint from accepting trailing slashes. ([\#6074](#6074)) - Return 403 on `/register/available` if registration has been disabled. ([\#6082](#6082)) - Explicitly log when a homeserver does not have the `trusted_key_servers` config field configured. ([\#6090](#6090)) - Add support for pruning old rows in `user_ips` table. ([\#6098](#6098)) Bugfixes -------- - Don't create broken room when `power_level_content_override.users` does not contain `creator_id`. ([\#5633](#5633)) - Fix database index so that different backup versions can have the same sessions. ([\#5857](#5857)) - Fix Synapse looking for config options `password_reset_failure_template` and `password_reset_success_template`, when they are actually `password_reset_template_failure_html`, `password_reset_template_success_html`. ([\#5863](#5863)) - Fix stack overflow when recovering an appservice which had an outage. ([\#5885](#5885)) - Fix error message which referred to `public_base_url` instead of `public_baseurl`. Thanks to @aaronraimist for the fix! ([\#5909](#5909)) - Fix 404 for thumbnail download when `dynamic_thumbnails` is `false` and the thumbnail was dynamically generated. Fix reported by rkfg. ([\#5915](#5915)) - Fix a cache-invalidation bug for worker-based deployments. ([\#5920](#5920)) - Fix admin API for listing media in a room not being available with an external media repo. ([\#5966](#5966)) - Fix list media admin API always returning an error. ([\#5967](#5967)) - Fix room and user stats tracking. ([\#5971](#5971), [\#5998](#5998), [\#6029](#6029)) - Return a `M_MISSING_PARAM` if `sid` is not provided to `/account/3pid`. ([\#5995](#5995)) - `federation_certificate_verification_whitelist` now will not cause `TypeErrors` to be raised (a regression in 1.3). Additionally, it now supports internationalised domain names in their non-canonical representation. ([\#5996](#5996)) - Only count real users when checking for auto-creation of auto-join room. ([\#6004](#6004)) - Ensure support users can be registered even if MAU limit is reached. ([\#6020](#6020)) - Fix bug where login error was shown incorrectly on SSO fallback login. ([\#6024](#6024)) - Fix bug in calculating the federation retry backoff period. ([\#6025](#6025)) - Prevent exceptions being logged when extremity-cleanup events fail due to lack of user consent to the terms of service. ([\#6053](#6053)) - Remove POST method from password-reset `submit_token` endpoint until we implement `submit_url` functionality. ([\#6056](#6056)) - Fix logcontext spam on non-Linux platforms. ([\#6059](#6059)) - Ensure query parameters in email validation links are URL-encoded. ([\#6063](#6063)) - Fix a bug which caused SAML attribute maps to be overridden by defaults. ([\#6069](#6069)) - Fix the logged number of updated items for the `users_set_deactivated_flag` background update. ([\#6092](#6092)) - Add `sid` to `next_link` for email validation. ([\#6097](#6097)) - Threepid validity checks on msisdns should not be dependent on `threepid_behaviour_email`. ([\#6104](#6104)) - Ensure that servers which are not configured to support email address verification do not offer it in the registration flows. ([\#6107](#6107)) Updates to the Docker image --------------------------- - Avoid changing `UID/GID` if they are already correct. ([\#5970](#5970)) - Provide `SYNAPSE_WORKER` envvar to specify python module. ([\#6058](#6058)) Improved Documentation ---------------------- - Convert documentation to markdown (from rst) ([\#5849](#5849)) - Update `INSTALL.md` to say that Python 2 is no longer supported. ([\#5953](#5953)) - Add developer documentation for using SAML2. ([\#6032](#6032)) - Add some notes on rolling back to v1.3.1. ([\#6049](#6049)) - Update the upgrade notes. ([\#6050](#6050)) Deprecations and Removals ------------------------- - Remove shared-secret registration from `/_matrix/client/r0/register` endpoint. Contributed by Awesome Technologies Innovationslabor GmbH. ([\#5877](#5877)) - Deprecate the `trusted_third_party_id_servers` option. ([\#5875](#5875)) Internal Changes ---------------- - Lay the groundwork for structured logging output. ([\#5680](#5680)) - Retry well-known lookup before the cache expires, giving a grace period where the remote well-known can be down but we still use the old result. ([\#5844](#5844)) - Remove log line for debugging issue #5407. ([\#5860](#5860)) - Refactor the Appservice scheduler code. ([\#5886](#5886)) - Compatibility with v2 Identity Service APIs other than /lookup. ([\#5892](#5892), [\#6013](#6013)) - Stop populating some unused tables. ([\#5893](#5893), [\#6047](#6047)) - Add missing index on `users_in_public_rooms` to improve the performance of directory queries. ([\#5894](#5894)) - Improve the logging when we have an error when fetching signing keys. ([\#5896](#5896)) - Add support for database engine-specific schema deltas, based on file extension. ([\#5911](#5911)) - Update Buildkite pipeline to use plugins instead of buildkite-agent commands. ([\#5922](#5922)) - Add link in sample config to the logging config schema. ([\#5926](#5926)) - Remove unnecessary parentheses in return statements. ([\#5931](#5931)) - Remove unused `jenkins/prepare_sytest.sh` file. ([\#5938](#5938)) - Move Buildkite pipeline config to the pipelines repo. ([\#5943](#5943)) - Remove unnecessary return statements in the codebase which were the result of a regex run. ([\#5962](#5962)) - Remove left-over methods from v1 registration API. ([\#5963](#5963)) - Cleanup event auth type initialisation. ([\#5975](#5975)) - Clean up dependency checking at setup. ([\#5989](#5989)) - Update OpenTracing docs to use the unified `trace` method. ([\#5776](#5776)) - Small refactor of function arguments and docstrings in` RoomMemberHandler`. ([\#6009](#6009)) - Remove unused `origin` argument on `FederationHandler.add_display_name_to_third_party_invite`. ([\#6010](#6010)) - Add a `failure_ts` column to the `destinations` database table. ([\#6016](#6016), [\#6072](#6072)) - Clean up some code in the retry logic. ([\#6017](#6017)) - Fix the structured logging tests stomping on the global log configuration for subsequent tests. ([\#6023](#6023)) - Clean up the sample config for SAML authentication. ([\#6064](#6064)) - Change mailer logging to reflect Synapse doesn't just do chat notifications by email now. ([\#6075](#6075)) - Move last-seen info into devices table. ([\#6089](#6089)) - Remove unused parameter to `get_user_id_by_threepid`. ([\#6099](#6099)) - Refactor the user-interactive auth handling. ([\#6105](#6105)) - Refactor code for calculating registration flows. ([\#6106](#6106))

Bumps `matrix-synapse` to version 1.4.0[1]. With this version the following changes in the matrix-synapse module were needed: * Removed `trusted_third_party_id_servers`: option is marked as deprecated and ignored by matrix-synapse[2]. * Added `account_threepid_delegates` options as replacement for 3rdparty server features[3]. * Added `redaction_retention_period` option to configure how long redacted options should be kept in the database. * Added `ma27` as maintainer for `matrix-synapse`. Co-Authored-By: Notkea <pacien@users.noreply.github.com> Co-authored-by: Maximilian Bosch <maximilian@mbosch.me> [1] https://matrix.org/blog/2019/10/03/synapse-1-4-0-released [2] matrix-org/synapse#5875 [3] matrix-org/synapse#5876

Bumps `matrix-synapse` to version 1.4.0[1]. With this version the following changes in the matrix-synapse module were needed: * Removed `trusted_third_party_id_servers`: option is marked as deprecated and ignored by matrix-synapse[2]. * Added `account_threepid_delegates` options as replacement for 3rdparty server features[3]. * Added `redaction_retention_period` option to configure how long redacted options should be kept in the database. * Added `ma27` as maintainer for `matrix-synapse`. Co-Authored-By: Notkea <pacien@users.noreply.github.com> Co-authored-by: Maximilian Bosch <maximilian@mbosch.me> [1] https://matrix.org/blog/2019/10/03/synapse-1-4-0-released [2] matrix-org/synapse#5875 [3] matrix-org/synapse#5876 (cherry picked from commit 3724223)

Bumps `matrix-synapse` to version 1.4.0[1]. With this version the following changes in the matrix-synapse module were needed: * Removed `trusted_third_party_id_servers`: option is marked as deprecated and ignored by matrix-synapse[2]. * Added `account_threepid_delegates` options as replacement for 3rdparty server features[3]. * Added `redaction_retention_period` option to configure how long redacted options should be kept in the database. * Added `ma27` as maintainer for `matrix-synapse`. Co-Authored-By: Notkea <pacien@users.noreply.github.com> Co-authored-by: Maximilian Bosch <maximilian@mbosch.me> [1] https://matrix.org/blog/2019/10/03/synapse-1-4-0-released [2] matrix-org/synapse#5875 [3] matrix-org/synapse#5876

Replace trust_identity_server_for_password_resets with account_threep…

09a3bb6

…id_delegate

anoadragon453 added z-privacy-sprint (Deprecated Label) phase:1 labels Aug 19, 2019

anoadragon453 requested a review from a team August 19, 2019 13:16

Add changelog

98191da

anoadragon453 mentioned this pull request Aug 19, 2019

[1/2] Allow homeservers to send registration emails | Sending the email #5835

Merged

1 task

erikjohnston suggested changes Aug 19, 2019

View reviewed changes

erikjohnston mentioned this pull request Aug 19, 2019

Remove trusted_third_party_id_servers functionality #5875

Merged

anoadragon453 and others added 7 commits August 19, 2019 16:24

Update synapse/config/emailconfig.py

52f1444

Co-Authored-By: Erik Johnston <erik@matrix.org>

Update synapse/handlers/identity.py

a148bba

Co-Authored-By: Erik Johnston <erik@matrix.org>

Update synapse/handlers/identity.py

5d9f48d

Co-Authored-By: Erik Johnston <erik@matrix.org>

Address review comments

3ba7304

Merge branch 'anoa/reg_email' into anoa/reg_email_account_threepid_de…

e65faf5

…legate * anoa/reg_email: Remove trusted_third_party_id_servers functionality (#5875)

lint

03a6eee

anoadragon453 requested a review from erikjohnston August 19, 2019 17:31

erikjohnston suggested changes Aug 20, 2019

View reviewed changes

anoadragon453 and others added 2 commits August 20, 2019 10:42

Update synapse/config/emailconfig.py

816fcd1

Co-Authored-By: Erik Johnston <erik@matrix.org>

Address review comments

0d2a024

anoadragon453 commented Aug 20, 2019

View reviewed changes

lint

f683e3b

anoadragon453 requested a review from erikjohnston August 20, 2019 13:40

erikjohnston suggested changes Aug 20, 2019

View reviewed changes

synapse/handlers/identity.py Outdated Show resolved Hide resolved

synapse/handlers/identity.py Outdated Show resolved Hide resolved

anoadragon453 and others added 3 commits August 20, 2019 15:10

Update synapse/handlers/identity.py

93bd5df

Co-Authored-By: Erik Johnston <erik@matrix.org>

Lift up id_server check

a94cf0d

Fix up msisdns

ed3ff55

anoadragon453 force-pushed the anoa/reg_email_account_threepid_delegate branch from e03c029 to ed3ff55 Compare August 20, 2019 15:57

anoadragon453 added 3 commits August 20, 2019 17:00

lint

e395133

Mark password resets disabled if behaviour is set to OFF

8aa1a4c

lint

4d92ad6

anoadragon453 added 2 commits August 20, 2019 17:13

Don't leak config info

a7f619d

Again

512d094

anoadragon453 commented Aug 20, 2019

View reviewed changes

synapse/rest/client/v2_alpha/account.py Show resolved Hide resolved

anoadragon453 requested a review from erikjohnston August 20, 2019 16:28

Warn about deprecated options

76ca096

erikjohnston approved these changes Aug 21, 2019

View reviewed changes

anoadragon453 added 2 commits August 21, 2019 16:38

Don't say MSISDN on email-based endpoints

37b0ce3

lint

00bca3b

anoadragon453 merged commit 7739f23 into anoa/reg_email Aug 21, 2019

anoadragon453 deleted the anoa/reg_email_account_threepid_delegate branch August 21, 2019 16:13

anoadragon453 mentioned this pull request Aug 28, 2019

Add flag in /versions for whether clients should send id_server params #5868

Merged

turt2live mentioned this pull request Aug 29, 2019

MSC2263: Give homeservers the ability to handle their own 3PID registrations/password resets matrix-org/matrix-spec-proposals#2263

Merged

This was referenced Sep 3, 2019

Set m.require_identity_server to always be False #5968

Closed

Allow Synapse to send registration emails + choose Synapse or an external server to handle 3pid validation #5987

Merged

3nprob mentioned this pull request Jul 27, 2022

Drop support for delegating email validation #13192

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Replace trust_identity_server_for_password_resets with account_threepid_delegate #5876

Replace trust_identity_server_for_password_resets with account_threepid_delegate #5876

anoadragon453 commented Aug 19, 2019 •

edited

Loading

erikjohnston Aug 19, 2019

anoadragon453 Aug 19, 2019

erikjohnston Aug 19, 2019 •

edited

Loading

anoadragon453 Aug 19, 2019

anoadragon453 Aug 19, 2019

anoadragon453 Aug 19, 2019

erikjohnston Aug 20, 2019

anoadragon453 Aug 20, 2019 •

edited

Loading

anoadragon453 Aug 20, 2019 •

edited

Loading

Replace trust_identity_server_for_password_resets with account_threepid_delegate #5876

Replace trust_identity_server_for_password_resets with account_threepid_delegate #5876

Conversation

anoadragon453 commented Aug 19, 2019 • edited Loading

erikjohnston Aug 19, 2019

Choose a reason for hiding this comment

anoadragon453 Aug 19, 2019

Choose a reason for hiding this comment

erikjohnston Aug 19, 2019 • edited Loading

Choose a reason for hiding this comment

anoadragon453 Aug 19, 2019

Choose a reason for hiding this comment

anoadragon453 Aug 19, 2019

Choose a reason for hiding this comment

anoadragon453 Aug 19, 2019

Choose a reason for hiding this comment

erikjohnston Aug 20, 2019

Choose a reason for hiding this comment

anoadragon453 Aug 20, 2019 • edited Loading

Choose a reason for hiding this comment

anoadragon453 Aug 20, 2019 • edited Loading

Choose a reason for hiding this comment

anoadragon453 commented Aug 19, 2019 •

edited

Loading

erikjohnston Aug 19, 2019 •

edited

Loading

anoadragon453 Aug 20, 2019 •

edited

Loading

anoadragon453 Aug 20, 2019 •

edited

Loading