X-Git-Url: https://git.nmode.ca/signal-cli/blobdiff_plain/00cda598c849ac0dd5301a961ee4113093b533c8..d690b35ed943b0210e3f483528b8c033fe2d667f:/lib/src/main/java/org/asamk/signal/manager/ProvisioningManagerImpl.java diff --git a/lib/src/main/java/org/asamk/signal/manager/ProvisioningManagerImpl.java b/lib/src/main/java/org/asamk/signal/manager/ProvisioningManagerImpl.java index 466286b0..e0f1084b 100644 --- a/lib/src/main/java/org/asamk/signal/manager/ProvisioningManagerImpl.java +++ b/lib/src/main/java/org/asamk/signal/manager/ProvisioningManagerImpl.java @@ -1,5 +1,5 @@ /* - Copyright (C) 2015-2021 AsamK and contributors + Copyright (C) 2015-2022 AsamK and contributors This program is free software: you can redistribute it and/or modify it under the terms of the GNU General Public License as published by @@ -16,9 +16,11 @@ */ package org.asamk.signal.manager; +import org.asamk.signal.manager.api.UserAlreadyExistsException; import org.asamk.signal.manager.config.ServiceConfig; import org.asamk.signal.manager.config.ServiceEnvironmentConfig; import org.asamk.signal.manager.storage.SignalAccount; +import org.asamk.signal.manager.storage.accounts.AccountsStore; import org.asamk.signal.manager.storage.identities.TrustNewIdentity; import org.asamk.signal.manager.util.KeyUtils; import org.slf4j.Logger; @@ -38,7 +40,7 @@ import java.net.URI; import java.util.concurrent.TimeoutException; import java.util.function.Consumer; -public class ProvisioningManagerImpl implements ProvisioningManager { +class ProvisioningManagerImpl implements ProvisioningManager { private final static Logger logger = LoggerFactory.getLogger(ProvisioningManagerImpl.class); @@ -46,6 +48,7 @@ public class ProvisioningManagerImpl implements ProvisioningManager { private final ServiceEnvironmentConfig serviceEnvironmentConfig; private final String userAgent; private final Consumer newManagerListener; + private final AccountsStore accountsStore; private final SignalServiceAccountManager accountManager; private final IdentityKeyPair tempIdentityKey; @@ -56,12 +59,14 @@ public class ProvisioningManagerImpl implements ProvisioningManager { PathConfig pathConfig, ServiceEnvironmentConfig serviceEnvironmentConfig, String userAgent, - final Consumer newManagerListener + final Consumer newManagerListener, + final AccountsStore accountsStore ) { this.pathConfig = pathConfig; this.serviceEnvironmentConfig = serviceEnvironmentConfig; this.userAgent = userAgent; this.newManagerListener = newManagerListener; + this.accountsStore = accountsStore; tempIdentityKey = KeyUtils.generateIdentityKeyPair(); registrationId = KeyHelper.generateRegistrationId(false); @@ -87,14 +92,26 @@ public class ProvisioningManagerImpl implements ProvisioningManager { } @Override - public String finishDeviceLink(String deviceName) throws IOException, TimeoutException, UserAlreadyExists { + public String finishDeviceLink(String deviceName) throws IOException, TimeoutException, UserAlreadyExistsException { var ret = accountManager.getNewDeviceRegistration(tempIdentityKey); var number = ret.getNumber(); + var aci = ret.getAci(); logger.info("Received link information from {}, linking in progress ...", number); - if (SignalAccount.userExists(pathConfig.dataPath(), number) && !canRelinkExistingAccount(number)) { - throw new UserAlreadyExists(number, SignalAccount.getFileName(pathConfig.dataPath(), number)); + var accountPath = accountsStore.getPathByAci(aci); + if (accountPath == null) { + accountPath = accountsStore.getPathByNumber(number); + } + if (accountPath != null + && SignalAccount.accountFileExists(pathConfig.dataPath(), accountPath) + && !canRelinkExistingAccount(accountPath)) { + throw new UserAlreadyExistsException(number, SignalAccount.getFileName(pathConfig.dataPath(), accountPath)); + } + if (accountPath == null) { + accountPath = accountsStore.addAccount(number, aci); + } else { + accountsStore.updateAccount(accountPath, number, aci); } var encryptedDeviceName = deviceName == null @@ -114,8 +131,9 @@ public class ProvisioningManagerImpl implements ProvisioningManager { SignalAccount account = null; try { account = SignalAccount.createOrUpdateLinkedAccount(pathConfig.dataPath(), + accountPath, number, - ret.getAci(), + aci, password, encryptedDeviceName, deviceId, @@ -126,7 +144,12 @@ public class ProvisioningManagerImpl implements ProvisioningManager { ManagerImpl m = null; try { - m = new ManagerImpl(account, pathConfig, serviceEnvironmentConfig, userAgent); + final var accountPathFinal = accountPath; + m = new ManagerImpl(account, + pathConfig, + (newNumber, newAci) -> accountsStore.updateAccount(accountPathFinal, newNumber, newAci), + serviceEnvironmentConfig, + userAgent); account = null; logger.debug("Refreshing pre keys"); @@ -161,10 +184,13 @@ public class ProvisioningManagerImpl implements ProvisioningManager { } } - private boolean canRelinkExistingAccount(final String number) throws IOException { + private boolean canRelinkExistingAccount(final String accountPath) throws IOException { final SignalAccount signalAccount; try { - signalAccount = SignalAccount.load(pathConfig.dataPath(), number, false, TrustNewIdentity.ON_FIRST_USE); + signalAccount = SignalAccount.load(pathConfig.dataPath(), + accountPath, + false, + TrustNewIdentity.ON_FIRST_USE); } catch (IOException e) { logger.debug("Account in use or failed to load.", e); return false; @@ -176,7 +202,11 @@ public class ProvisioningManagerImpl implements ProvisioningManager { return false; } - final var m = new ManagerImpl(signalAccount, pathConfig, serviceEnvironmentConfig, userAgent); + final var m = new ManagerImpl(signalAccount, + pathConfig, + (newNumber, newAci) -> accountsStore.updateAccount(accountPath, newNumber, newAci), + serviceEnvironmentConfig, + userAgent); try (m) { m.checkAccountState(); } catch (AuthorizationFailedException ignored) {