Chapril/xmpp.chapril.org-conversejs
Chapril/xmpp.chapril.org-conversejs
24
0
Fork 0
Code Issues Pull Requests Projects Releases Activity

Bugfix. Avoid calling this.authenticate twice in one method

Browse Source
This commit is contained in:
JC Brand 2021-12-11 12:28:04 +01:00
parent 8ad097f23f
commit e9ae5306e3
1 changed files with 10 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
src/plugins/controlbox/loginform.js
View File

@ -29,13 +29,18 @@ class LoginForm extends CustomElement {
if (api.settings.get('bosh_service_url') ||
api.settings.get('websocket_url') ||
this.model.get('show_connection_url_input')) {
this.authenticate(ev);
}
await this.discoverConnectionMethods(ev);
if (api.settings.get('bosh_service_url') || api.settings.get('websocket_url')) {
// The connection class will still try to discover XEP-0156 connection methods
this.authenticate(ev);
} else {
this.model.set('show_connection_url_input', true);
// We don't have a connection URL available, so we try here to discover
// XEP-0156 connection methods now, and if not found we present the user
// with the option to enter their own connection URL
await this.discoverConnectionMethods(ev);
if (api.settings.get('bosh_service_url') || api.settings.get('websocket_url')) {
this.authenticate(ev);
} else {
this.model.set('show_connection_url_input', true);
}
}
}