OpenCloud
/
element-web
mirror of https://github.com/vector-im/element-web
1
0
Fork 0
Code Issues Releases Wiki Activity

Remove 3PID binding during registration 

This disables 3PID binding at registration time, so users won't be discoverable
by 3PID by default. Instead, new discovery controls in settings allow you to
opt-in.

Fixes https://github.com/vector-im/riot-web/issues/10424
pull/21833/head
J. Ryan Stinnett 2019-08-08 15:52:54 +01:00
parent 07d6fa7fdb
commit 38e64ce966
1 changed files with 2 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/components/structures/auth/Registration.js
View File

@ -403,14 +403,9 @@ module.exports = React.createClass({
// clicking the email link. // clicking the email link.
let inhibitLogin = Boolean(this.state.formVals.email); let inhibitLogin = Boolean(this.state.formVals.email);
// Only send the bind params if we're sending username / pw params // Only send inhibitLogin if we're sending username / pw params
// (Since we need to send no params at all to use the ones saved in the // (Since we need to send no params at all to use the ones saved in the
// session). // session).
const bindThreepids = this.state.formVals.password ? {
email: true,
msisdn: true,
} : {};
// Likewise inhibitLogin
if (!this.state.formVals.password) inhibitLogin = null; if (!this.state.formVals.password) inhibitLogin = null;
return this.state.matrixClient.register( return this.state.matrixClient.register(
@ -418,7 +413,7 @@ module.exports = React.createClass({
this.state.formVals.password, this.state.formVals.password,
undefined, // session id: included in the auth dict already undefined, // session id: included in the auth dict already
auth, auth,
bindThreepids, null,
null, null,
inhibitLogin, inhibitLogin,
); );