imterah/element-portable
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Prompt for terms of service on integration manager changes

Browse source 
Part of https://github.com/vector-im/riot-web/issues/10539
This commit is contained in:
Travis Ralston 2019-08-15 13:28:23 -06:00
parent ded2297523
commit 27504e1578
5 changed files with 122 additions and 37 deletions
Download patch file Download diff file Expand all files Collapse all files

19
src/integrations/IntegrationManagers.js
View file

@ -117,7 +117,8 @@ export class IntegrationManagers {
}
/**
* Attempts to discover an integration manager using only its name.
* Attempts to discover an integration manager using only its name. This will not validate that
* the integration manager is functional - that is the caller's responsibility.
* @param {string} domainName The domain name to look up.
* @returns {Promise<IntegrationManagerInstance>} Resolves to an integration manager instance,
* or null if none was found.
@ -153,20 +154,12 @@ export class IntegrationManagers {
// All discovered managers are per-user managers
const manager = new IntegrationManagerInstance(KIND_ACCOUNT, widget["data"]["api_url"], widget["url"]);
console.log("Got integration manager response, checking for responsiveness");
console.log("Got an integration manager (untested)");
// Test the manager
const client = manager.getScalarClient();
try {
// not throwing an error is a success here
await client.connect();
} catch (e) {
console.error(e);
console.warn("Integration manager failed liveliness check");
return null;
}
// We don't test the manager because the caller may need to do extra
// checks or similar with it. For instance, they may need to deal with
// terms of service or want to call something particular.
console.log("Integration manager is alive and functioning");
return manager;
}
}