diff --git a/src/components/views/settings/SetIdServer.js b/src/components/views/settings/SetIdServer.js
index a718d87fa6..e3d6d18c5b 100644
--- a/src/components/views/settings/SetIdServer.js
+++ b/src/components/views/settings/SetIdServer.js
@@ -137,7 +137,12 @@ export default class SetIdServer extends React.Component {
MatrixClientPeg.get().setAccountData("m.identity_server", {
base_url: fullUrl,
});
- this.setState({idServer: '', busy: false, error: null});
+ this.setState({
+ busy: false,
+ error: null,
+ currentClientIdServer: fullUrl,
+ idServer: '',
+ });
};
_checkIdServer = async (e) => {
@@ -157,14 +162,34 @@ export default class SetIdServer extends React.Component {
const authClient = new IdentityAuthClient(fullUrl);
await authClient.getAccessToken();
+ let save = true;
+
// Double check that the identity server even has terms of service.
const terms = await MatrixClientPeg.get().getTerms(SERVICE_TYPES.IS, fullUrl);
if (!terms || !terms["policies"] || Object.keys(terms["policies"]).length <= 0) {
- this._showNoTermsWarning(fullUrl);
- return;
+ save &= await this._showNoTermsWarning(fullUrl);
}
- this._saveIdServer(fullUrl);
+ // Show a general warning, possibly with details about any bound
+ // 3PIDs that would be left behind.
+ if (this.state.currentClientIdServer) {
+ save &= await this._showServerChangeWarning({
+ title: _t("Change identity server"),
+ unboundMessage: _t(
+ "Disconnect from the identity server and " +
+ "connect to instead?", {},
+ {
+ current: sub => {abbreviateUrl(this.state.currentClientIdServer)},
+ new: sub => {abbreviateUrl(this.state.idServer)},
+ },
+ ),
+ button: _t("Continue"),
+ });
+ }
+
+ if (save) {
+ this._saveIdServer(fullUrl);
+ }
} catch (e) {
console.error(e);
if (e.cors === "rejected" || e.httpStatus === 404) {
@@ -179,73 +204,80 @@ export default class SetIdServer extends React.Component {
checking: false,
error: errStr,
currentClientIdServer: MatrixClientPeg.get().getIdentityServerUrl(),
- idServer: this.state.idServer,
});
};
_showNoTermsWarning(fullUrl) {
const QuestionDialog = sdk.getComponent("views.dialogs.QuestionDialog");
- Modal.createTrackedDialog('No Terms Warning', '', QuestionDialog, {
- title: _t("Identity server has no terms of service"),
- description: (
-
-
- {_t("The identity server you have chosen does not have any terms of service.")}
-
-
- {_t("Only continue if you trust the owner of the server.")}
-
-
- ),
- button: _t("Continue"),
- onFinished: async (confirmed) => {
- if (!confirmed) return;
- this._saveIdServer(fullUrl);
- },
+ return new Promise(resolve => {
+ Modal.createTrackedDialog('No Terms Warning', '', QuestionDialog, {
+ title: _t("Identity server has no terms of service"),
+ description: (
+
+
+ {_t("The identity server you have chosen does not have any terms of service.")}
+
+
+ {_t("Only continue if you trust the owner of the server.")}
+
+
+ ),
+ button: _t("Continue"),
+ onFinished: resolve,
+ });
});
}
_onDisconnectClicked = async () => {
this.setState({disconnectBusy: true});
try {
- const threepids = await getThreepidBindStatus(MatrixClientPeg.get());
-
- const boundThreepids = threepids.filter(tp => tp.bound);
- let message;
- if (boundThreepids.length) {
- message = _t(
- "You are currently sharing email addresses or phone numbers on the identity " +
- "server . You will need to reconnect to to stop " +
- "sharing them.", {},
- {
- idserver: sub => {abbreviateUrl(this.state.currentClientIdServer)},
- // XXX: https://github.com/vector-im/riot-web/issues/9086
- idserver2: sub => {abbreviateUrl(this.state.currentClientIdServer)},
- },
- );
- } else {
- message = _t(
+ const confirmed = await this._showServerChangeWarning({
+ title: _t("Disconnect identity server"),
+ unboundMessage: _t(
"Disconnect from the identity server ?", {},
{idserver: sub => {abbreviateUrl(this.state.currentClientIdServer)}},
- );
- }
-
- const QuestionDialog = sdk.getComponent("dialogs.QuestionDialog");
- Modal.createTrackedDialog('Identity Server Disconnect Warning', '', QuestionDialog, {
- title: _t("Disconnect Identity Server"),
- description: message,
+ ),
button: _t("Disconnect"),
- onFinished: (confirmed) => {
- if (confirmed) {
- this._disconnectIdServer();
- }
- },
});
+ if (confirmed) {
+ this._disconnectIdServer();
+ }
} finally {
this.setState({disconnectBusy: false});
}
};
+ async _showServerChangeWarning({ title, unboundMessage, button }) {
+ const threepids = await getThreepidBindStatus(MatrixClientPeg.get());
+
+ const boundThreepids = threepids.filter(tp => tp.bound);
+ let message;
+ if (boundThreepids.length) {
+ message = _t(
+ "You are currently sharing email addresses or phone numbers on the identity " +
+ "server . You will need to reconnect to to stop " +
+ "sharing them.", {},
+ {
+ idserver: sub => {abbreviateUrl(this.state.currentClientIdServer)},
+ // XXX: https://github.com/vector-im/riot-web/issues/9086
+ idserver2: sub => {abbreviateUrl(this.state.currentClientIdServer)},
+ },
+ );
+ } else {
+ message = unboundMessage;
+ }
+
+ const QuestionDialog = sdk.getComponent("dialogs.QuestionDialog");
+ return new Promise(resolve => {
+ Modal.createTrackedDialog('Identity Server Bound Warning', '', QuestionDialog, {
+ title,
+ description: message,
+ button,
+ onFinished: resolve,
+ });
+ });
+ }
+
_disconnectIdServer = () => {
// Account data change will update localstorage, client, etc through dispatcher
MatrixClientPeg.get().setAccountData("m.identity_server", {
diff --git a/src/i18n/strings/en_EN.json b/src/i18n/strings/en_EN.json
index dd0894b813..f31fcc7157 100644
--- a/src/i18n/strings/en_EN.json
+++ b/src/i18n/strings/en_EN.json
@@ -554,14 +554,16 @@
"Not a valid Identity Server (status code %(code)s)": "Not a valid Identity Server (status code %(code)s)",
"Could not connect to Identity Server": "Could not connect to Identity Server",
"Checking server": "Checking server",
+ "Change identity server": "Change identity server",
+ "Disconnect from the identity server and connect to instead?": "Disconnect from the identity server and connect to instead?",
"Terms of service not accepted or the identity server is invalid.": "Terms of service not accepted or the identity server is invalid.",
"Identity server has no terms of service": "Identity server has no terms of service",
"The identity server you have chosen does not have any terms of service.": "The identity server you have chosen does not have any terms of service.",
"Only continue if you trust the owner of the server.": "Only continue if you trust the owner of the server.",
- "You are currently sharing email addresses or phone numbers on the identity server . You will need to reconnect to to stop sharing them.": "You are currently sharing email addresses or phone numbers on the identity server . You will need to reconnect to to stop sharing them.",
+ "Disconnect identity server": "Disconnect identity server",
"Disconnect from the identity server ?": "Disconnect from the identity server ?",
- "Disconnect Identity Server": "Disconnect Identity Server",
"Disconnect": "Disconnect",
+ "You are currently sharing email addresses or phone numbers on the identity server . You will need to reconnect to to stop sharing them.": "You are currently sharing email addresses or phone numbers on the identity server . You will need to reconnect to to stop sharing them.",
"Identity Server (%(server)s)": "Identity Server (%(server)s)",
"You are currently using to discover and be discoverable by existing contacts you know. You can change your identity server below.": "You are currently using to discover and be discoverable by existing contacts you know. You can change your identity server below.",
"If you don't want to use to discover and be discoverable by existing contacts you know, enter another identity server below.": "If you don't want to use to discover and be discoverable by existing contacts you know, enter another identity server below.",