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

Simplify attemptNonPreboundSession

Browse Source 
- Split between `login` auth and the rest (`external`, `anonymous`, `prebind`)
- It doesn't make sense to pass along credentials when not using `login` auth so avoid that flow.
- It's not necessary to pass the `reconnecting` flag to `attemptNonPreboundSession`
This commit is contained in:
JC Brand 2019-07-23 08:08:25 +02:00
parent 30ad84d386
commit e4eafe9e91
2 changed files with 25 additions and 29 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
dev.html
View File

@ -25,7 +25,6 @@
// 'prosody@conference.prosody.im',
// 'jdev@conference.jabber.org'
// ],
// bosh_service_url: 'http://chat.example.org:5280/http-bind/',
websocket_url: 'wss://conversejs.org/xmpp-websocket',
bosh_service_url: 'https://conversejs.org/http-bind/', // Please use this connection manager only for testing purposes
view_mode: 'fullscreen',

47
src/headless/converse-core.js
View File

@ -642,6 +642,14 @@ async function getLoginCredentials () {
return credentials;
}
async function getLoginCredentialsFromBrowser () {
const creds = await navigator.credentials.get({'password': true});
if (creds && creds.type == 'password' && u.isValidJID(creds.id)) {
await setUserJID(creds.id);
return {'jid': creds.id, 'password': creds.password};
}
}
function unregisterGlobalEventHandlers () {
document.removeEventListener("visibilitychange", _converse.saveWindowState);
@ -1213,34 +1221,21 @@ _converse.initialize = async function (settings, callback) {
}
});
this.attemptNonPreboundSession = async function (credentials, reconnecting) {
this.attemptNonPreboundSession = async function (credentials) {
if (_converse.authentication === _converse.LOGIN) {
if (credentials) {
this.autoLogin(credentials);
} else if (this.auto_login) {
if (this.credentials_url && _converse.authentication === _converse.LOGIN) {
const data = await getLoginCredentials();
this.autoLogin(data);
} else if (!this.jid) {
throw new Error(
"attemptNonPreboundSession: If you use auto_login, "+
"you also need to give either a jid value (and if "+
"applicable a password) or you need to pass in a URL "+
"from where the username and password can be fetched "+
"(via credentials_url)."
);
} else {
// Could be ANONYMOUS or EXTERNAL or manual passing in of JID and password
this.autoLogin();
}
} else if (reconnecting) {
} else if (_converse.jid && (_converse.password || _converse.connection.pass)) {
this.autoLogin();
} else if (_converse.credentials_url) {
this.autoLogin(await getLoginCredentials());
} else if (!_converse.isTestEnv() && window.PasswordCredential) {
const creds = await navigator.credentials.get({'password': true});
if (creds && creds.type == 'password' && u.isValidJID(creds.id)) {
await setUserJID(creds.id);
this.autoLogin({'jid': creds.id, 'password': creds.password});
_converse.autoLogin(await getLoginCredentialsFromBrowser());
} else {
throw new Error("attemptNonPreboundSession: Could not find any credentials to log you in with!");
}
} else {
this.autoLogin(); // Could be ANONYMOUS or EXTERNAL
}
};
@ -1464,7 +1459,9 @@ _converse.api = {
*/
async login (jid, password, reconnecting) {
if (_converse.api.connection.isType('bosh')) {
if (reconnecting && _converse.prebind_url) {
if (reconnecting &&
_converse.authentication === _converse.PREBIND &&
_converse.prebind_url) {
return _converse.startNewBOSHSession();
} else if (await _converse.restoreBOSHSession()) {
return;
@ -1476,7 +1473,7 @@ _converse.api = {
}
password = password || _converse.password;
const credentials = (jid && password) ? { jid, password } : null;
_converse.attemptNonPreboundSession(credentials, reconnecting);
_converse.attemptNonPreboundSession(credentials);
},
/**