Merge pull request #3949 from matrix-org/foldleft/12079-fix-warnings-on-self

Fix issue where we don't notice if our own devices shouldn't be trusted
This commit is contained in:
Zoe 2020-01-28 10:16:27 +00:00 committed by GitHub
commit 06f7842afa
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 2 additions and 2 deletions

View file

@ -811,7 +811,7 @@ export default createReactClass({
debuglog("e2e verified", verified, "unverified", unverified); debuglog("e2e verified", verified, "unverified", unverified);
/* Check all verified user devices. */ /* Check all verified user devices. */
for (const userId of verified) { for (const userId of [...verified, cli.getUserId()]) {
const devices = await cli.getStoredDevicesForUser(userId); const devices = await cli.getStoredDevicesForUser(userId);
const anyDeviceNotVerified = devices.some(({deviceId}) => { const anyDeviceNotVerified = devices.some(({deviceId}) => {
return !cli.checkDeviceTrust(userId, deviceId).isVerified(); return !cli.checkDeviceTrust(userId, deviceId).isVerified();

View file

@ -166,7 +166,7 @@ export default createReactClass({
}); });
/* Check all verified user devices. */ /* Check all verified user devices. */
for (const userId of verified) { for (const userId of [...verified, cli.getUserId()]) {
const devices = await cli.getStoredDevicesForUser(userId); const devices = await cli.getStoredDevicesForUser(userId);
const allDevicesVerified = devices.every(({deviceId}) => { const allDevicesVerified = devices.every(({deviceId}) => {
return cli.checkDeviceTrust(userId, deviceId).isVerified(); return cli.checkDeviceTrust(userId, deviceId).isVerified();