sync on wallet lock before checking if wallet is behind

This commit is contained in:
woodser 2024-08-05 12:23:31 -04:00
parent 3dfaa2fc52
commit 443c2f4cdb

View file

@ -2412,6 +2412,7 @@ public abstract class Trade extends XmrWalletBase implements Tradable, Model {
private void syncWallet(boolean pollWallet) { private void syncWallet(boolean pollWallet) {
MoneroRpcConnection sourceConnection = xmrConnectionService.getConnection(); MoneroRpcConnection sourceConnection = xmrConnectionService.getConnection();
try { try {
synchronized (walletLock) {
if (getWallet() == null) throw new RuntimeException("Cannot sync trade wallet because it doesn't exist for " + getClass().getSimpleName() + ", " + getId()); if (getWallet() == null) throw new RuntimeException("Cannot sync trade wallet because it doesn't exist for " + getClass().getSimpleName() + ", " + getId());
if (getWallet().getDaemonConnection() == null) throw new RuntimeException("Cannot sync trade wallet because it's not connected to a Monero daemon for " + getClass().getSimpleName() + ", " + getId()); if (getWallet().getDaemonConnection() == null) throw new RuntimeException("Cannot sync trade wallet because it's not connected to a Monero daemon for " + getClass().getSimpleName() + ", " + getId());
if (isWalletBehind()) { if (isWalletBehind()) {
@ -2420,6 +2421,7 @@ public abstract class Trade extends XmrWalletBase implements Tradable, Model {
syncWalletIfBehind(); syncWalletIfBehind();
log.info("Done syncing wallet for {} {} in {} ms", getClass().getSimpleName(), getShortId(), System.currentTimeMillis() - startTime); log.info("Done syncing wallet for {} {} in {} ms", getClass().getSimpleName(), getShortId(), System.currentTimeMillis() - startTime);
} }
}
// apply tor after wallet synced depending on configuration // apply tor after wallet synced depending on configuration
if (!wasWalletSynced) { if (!wasWalletSynced) {
@ -2627,8 +2629,8 @@ public abstract class Trade extends XmrWalletBase implements Tradable, Model {
} }
private void syncWalletIfBehind() { private void syncWalletIfBehind() {
if (isWalletBehind()) {
synchronized (walletLock) { synchronized (walletLock) {
if (isWalletBehind()) {
syncWithProgress(); syncWithProgress();
} }
} }