@bitpay-labs/bitcore-wallet-service 11.11.0 → 11.11.2

package/ts_build/src/lib/server.js

@@ -61,7 +61,6 @@ const sardine_1 = require("../externalservices/sardine");
 const simplex_1 = require("../externalservices/simplex");
 const thorswap_1 = require("../externalservices/thorswap");
 const transak_1 = require("../externalservices/transak");
-const wyre_1 = require("../externalservices/wyre");
 const serverMessages_1 = require("../serverMessages");
 const bchaddresstranslator_1 = require("./bchaddresstranslator");
 const blockchainexplorer_1 = require("./blockchainexplorer");
@@ -145,7 +144,6 @@ class WalletService {
             simplex: new simplex_1.SimplexService(),
             thorswap: new thorswap_1.ThorswapService(),
             transak: new transak_1.TransakService(),
-            wyre: new wyre_1.WyreService(),
             coinGecko: new coingecko_1.CoinGeckoService(this.storage),
         };
     }
@@ -643,25 +641,25 @@ class WalletService {
     getStatus(opts, cb) {
         opts = opts || {};
         const status = {};
-        async.parallel([
-            next => {
-                this.getWallet({}, (err, wallet) => {
-                    if (err)
-                        return next(err);
-                    if (this._upgradeNeeded(exports.UPGRADES.SOL_bwc_$lt_10_10_12, wallet)) {
-                        return next(errordefinitions_1.Errors.UPGRADE_NEEDED);
-                    }
+        this.getWallet({}, (err, fetchedWallet) => {
+            if (err)
+                return cb(err);
+            const copyOfWallet = () => model_1.Wallet.fromObj(fetchedWallet.toObject());
+            const omit = (obj, keys) => Object.fromEntries(Object.entries(obj).filter(([key]) => !keys.includes(key)));
+            async.parallel([
+                next => {
+                    let wallet = copyOfWallet();
                     const walletExtendedKeys = ['publicKeyRing', 'pubKey', 'addressManager'];
                     const copayerExtendedKeys = ['xPubKey', 'requestPubKey', 'signature', 'addressManager', 'customData'];
                     wallet.copayers = (wallet.copayers || []).map(copayer => {
                         if (copayer.id == this.copayerId)
                             return copayer;
-                        return _.omit(copayer, 'customData');
+                        return omit(copayer, ['customData']);
                     });
                     if (!opts.includeExtendedInfo) {
-                        wallet = _.omit(wallet, walletExtendedKeys);
+                        wallet = omit(wallet, walletExtendedKeys);
                         wallet.copayers = (wallet.copayers || []).map(copayer => {
-                            return _.omit(copayer, copayerExtendedKeys);
+                            return omit(copayer, copayerExtendedKeys);
                         });
                     }
                     status.wallet = wallet;
@@ -672,43 +670,42 @@ class WalletService {
                         status.serverMessage = (0, deprecated_serverMessages_1.serverMessages)(wallet, this.appName, this.appVersion);
                     }
                     next();
-                });
-            },
-            next => {
-                opts.wallet = status.wallet;
-                this.getBalance(opts, (err, balance) => {
-                    if (opts.includeExtendedInfo) {
-                        if (err && err.code != 'WALLET_NEED_SCAN') {
+                },
+                next => {
+                    this.getBalance({ ...opts, wallet: copyOfWallet() }, (err, balance) => {
+                        if (opts.includeExtendedInfo) {
+                            if (err && err.code != 'WALLET_NEED_SCAN') {
+                                return next(err);
+                            }
+                        }
+                        else if (err) {
                             return next(err);
                         }
-                    }
-                    else if (err) {
-                        return next(err);
-                    }
-                    status.balance = balance;
-                    next();
-                });
-            },
-            next => {
-                this.getPendingTxs(opts, (err, pendingTxps) => {
-                    if (err)
-                        return next(err);
-                    status.pendingTxps = pendingTxps;
-                    next();
-                });
-            },
-            next => {
-                this.getPreferences({}, (err, preferences) => {
-                    if (err)
-                        return next(err);
-                    status.preferences = preferences;
-                    next();
-                });
-            }
-        ], err => {
-            if (err)
-                return cb(err);
-            return cb(null, status);
+                        status.balance = balance;
+                        next();
+                    });
+                },
+                next => {
+                    this.getPendingTxs({ ...opts, wallet: copyOfWallet() }, (err, pendingTxps) => {
+                        if (err)
+                            return next(err);
+                        status.pendingTxps = pendingTxps;
+                        next();
+                    });
+                },
+                next => {
+                    this.getPreferences({}, (err, preferences) => {
+                        if (err)
+                            return next(err);
+                        status.preferences = preferences;
+                        next();
+                    });
+                }
+            ], err => {
+                if (err)
+                    return cb(err);
+                return cb(null, status);
+            });
         });
     }
     _verifySignature(text, signature, pubkey) {
@@ -1138,10 +1135,11 @@ class WalletService {
     _canCreateAddress(ignoreMaxGap, cb) {
         if (ignoreMaxGap)
             return cb(null, true);
-        this.storage.fetchAddresses(this.walletId, (err, addresses) => {
+        const newest = { noChange: true, reverse: true, limit: Defaults.MAX_MAIN_ADDRESS_GAP };
+        this.storage.fetchAddressesPage(this.walletId, newest, (err, addresses) => {
             if (err)
                 return cb(err);
-            const latestAddresses = addresses.filter(x => !x.isChange).slice(-Defaults.MAX_MAIN_ADDRESS_GAP);
+            const latestAddresses = addresses.reverse();
             if (latestAddresses.length < Defaults.MAX_MAIN_ADDRESS_GAP ||
                 latestAddresses.some(a => a.hasActivity))
                 return cb(null, true);
@@ -1214,7 +1212,7 @@ class WalletService {
             }, true);
         };
         const getFirstAddress = (wallet, cb) => {
-            this.storage.fetchAddresses(this.walletId, (err, addresses) => {
+            this.storage.fetchAddressesPage(this.walletId, { limit: 1 }, (err, addresses) => {
                 if (err)
                     return cb(err);
                 if (addresses?.length) {
@@ -1259,21 +1257,10 @@ class WalletService {
     }
     getAddresses(opts, cb) {
         opts = opts || {};
-        this.storage.fetchAddresses(this.walletId, (err, addresses) => {
+        const { noChange, addresses: only, reverse, skip, limit } = opts;
+        this.storage.fetchAddressesPage(this.walletId, { noChange, addresses: only, reverse, skip, limit }, (err, addresses) => {
             if (err)
                 return cb(err);
-            if (opts.noChange) {
-                addresses = addresses.filter(a => !a.isChange);
-            }
-            if (opts.addresses) {
-                addresses = addresses.filter(a => opts.addresses.includes(a.address));
-            }
-            if (opts.reverse)
-                addresses.reverse();
-            if (opts.skip > 0)
-                addresses = addresses.slice(opts.skip);
-            if (opts.limit > 0)
-                addresses = addresses.slice(0, opts.limit);
             this.getWallet({}, (err, wallet) => {
                 if (err)
                     return cb(err);
@@ -1329,7 +1316,7 @@ class WalletService {
         let allAddresses, allUtxos, utxoIndex, bc, wallet, blockchainHeight;
         async.series([
             next => {
-                this.getWallet({}, (err, w) => {
+                const useWallet = (err, w) => {
                     if (err)
                         return next(err);
                     wallet = w;
@@ -1339,7 +1326,10 @@ class WalletService {
                     if (!bc)
                         return cb(new Error('Could not get blockchain explorer instance'));
                     return next();
-                });
+                };
+                if (opts.wallet)
+                    return useWallet(null, opts.wallet);
+                this.getWallet({}, useWallet);
             },
             next => {
                 if (Array.isArray(opts.addresses)) {
@@ -1422,7 +1412,7 @@ class WalletService {
             },
             next => {
                 utxoIndex = _.keyBy(allUtxos, utxoKey);
-                this.getPendingTxs({ numberFormat: opts.numberFormat }, (err, txps) => {
+                this.getPendingTxs({ numberFormat: opts.numberFormat, wallet }, (err, txps) => {
                     if (err)
                         return next(err);
                     const lockedInputs = txps.flatMap(t => t.inputs).map(utxoKey);
@@ -1456,7 +1446,7 @@ class WalletService {
                     });
                     logger_1.default.debug(`Got ${allUtxos.length} usable UTXOs`);
                     return next();
-                });
+                }, wallet);
             },
             next => {
                 const addressToPath = _.keyBy(allAddresses, 'address');
@@ -1468,6 +1458,7 @@ class WalletService {
                     }
                     utxo.path = addressToPath[utxo.address].path;
                     utxo.publicKeys = addressToPath[utxo.address].publicKeys;
+                    utxo.isChange = addressToPath[utxo.address].isChange;
                 }
                 return next();
             }
@@ -1817,12 +1808,42 @@ class WalletService {
         return null;
     }
     _validateAndSanitizeTxOpts(wallet, opts, cb) {
+        const gasFeeFields = ['gasPrice', 'maxGasFee', 'priorityGasFee'];
+        const hasGasFeeOverride = gasFeeFields.some(key => opts[key] != null);
+        if (hasGasFeeOverride) {
+            if (!Constants.EVM_CHAINS[wallet.chain.toUpperCase()]) {
+                return cb(new clienterror_1.ClientError('Gas fee overrides are only supported for EVM chains'));
+            }
+            for (const key of gasFeeFields) {
+                if (opts[key] != null && (!Number.isSafeInteger(opts[key]) || opts[key] <= 0)) {
+                    return cb(new clienterror_1.ClientError(`${key} must be a positive safe integer in wei`));
+                }
+            }
+            if (opts.gasPrice != null && Number(opts.txType || 0) !== 0) {
+                return cb(new clienterror_1.ClientError('gasPrice requires a type 0 transaction'));
+            }
+            if ((opts.maxGasFee != null || opts.priorityGasFee != null) && Number(opts.txType) !== 2) {
+                return cb(new clienterror_1.ClientError('maxGasFee and priorityGasFee require a type 2 transaction'));
+            }
+            if ((opts.maxGasFee != null) !== (opts.priorityGasFee != null)) {
+                return cb(new clienterror_1.ClientError('maxGasFee and priorityGasFee must be supplied together'));
+            }
+            if (opts.priorityGasFee > opts.maxGasFee) {
+                return cb(new clienterror_1.ClientError('priorityGasFee must not exceed maxGasFee'));
+            }
+            if (opts.feePerKb != null || opts.fee != null) {
+                return cb(new clienterror_1.ClientError('Gas fee overrides cannot be combined with feePerKb or fee'));
+            }
+            if (opts.sendMax) {
+                return cb(new clienterror_1.ClientError('Gas fee overrides are not supported with sendMax'));
+            }
+        }
         async.series([
             next => {
                 const feeArgs = boolToNum(!!opts.feeLevel) + boolToNum(Utils.isNumber(opts.feePerKb)) + boolToNum(Utils.isNumber(opts.fee));
                 if (feeArgs > 1)
                     return next(new clienterror_1.ClientError('Only one of feeLevel/feePerKb/fee can be specified'));
-                if (feeArgs == 0) {
+                if (feeArgs == 0 && !hasGasFeeOverride) {
                     opts.feeLevel = 'normal';
                 }
                 const feeLevels = Defaults.FEE_LEVELS[wallet.chain];
@@ -1830,6 +1851,9 @@ class WalletService {
                     if (!feeLevels.some(lvl => lvl.name === opts.feeLevel))
                         return next(new clienterror_1.ClientError('Invalid fee level. Valid values are ' + feeLevels.map(lvl => lvl.name).join(', ')));
                 }
+                if (hasGasFeeOverride) {
+                    delete opts.feeLevel;
+                }
                 const error = index_1.ChainService.checkUtxos(wallet.chain, opts);
                 if (error) {
                     return next(new clienterror_1.ClientError('fee can only be set when inputs are specified'));
@@ -2467,39 +2491,47 @@ class WalletService {
         });
     }
     getTx(opts, cb) {
-        this.storage.fetchTx(this.walletId, opts.txProposalId, (err, txp) => {
+        this.storage.fetchWallet(this.walletId, (err, wallet) => {
             if (err)
                 return cb(err);
-            if (!txp)
-                return cb(errordefinitions_1.Errors.TX_NOT_FOUND);
-            if (!txp.txid)
-                return cb(null, txp);
-            this.storage.fetchTxNote(this.walletId, txp.txid, (err, note) => {
-                if (err) {
-                    this.logw(`Error fetching tx note for ${txp.txid}:`, err);
-                }
-                txp.note = note;
-                return cb(null, txp);
-            });
+            this.storage.fetchTx(this.walletId, opts.txProposalId, (err, txp) => {
+                if (err)
+                    return cb(err);
+                if (!txp)
+                    return cb(errordefinitions_1.Errors.TX_NOT_FOUND);
+                if (!txp.txid)
+                    return cb(null, txp);
+                this.storage.fetchTxNote(this.walletId, txp.txid, (err, note) => {
+                    if (err) {
+                        this.logw(`Error fetching tx note for ${txp.txid}:`, err);
+                    }
+                    txp.note = note;
+                    return cb(null, txp);
+                }, wallet);
+            }, wallet);
         });
     }
     getTxByHash(opts, cb) {
         if (!checkRequired(opts, 'txid', cb))
             return;
-        this.storage.fetchTxByHashForWallet(this.walletId, opts.txid, (err, txp) => {
+        this.storage.fetchWallet(this.walletId, (err, wallet) => {
             if (err)
                 return cb(err);
-            if (!txp)
-                return cb(errordefinitions_1.Errors.TX_NOT_FOUND);
-            if (!txp.txid)
-                return cb(null, txp);
-            this.storage.fetchTxNote(this.walletId, txp.txid, (err, note) => {
-                if (err) {
-                    this.logw(`Error fetching tx note for ${txp.txid}:`, err);
-                }
-                txp.note = note;
-                return cb(null, txp);
-            });
+            this.storage.fetchTxByHashForWallet(this.walletId, opts.txid, (err, txp) => {
+                if (err)
+                    return cb(err);
+                if (!txp)
+                    return cb(errordefinitions_1.Errors.TX_NOT_FOUND);
+                if (!txp.txid)
+                    return cb(null, txp);
+                this.storage.fetchTxNote(this.walletId, txp.txid, (err, note) => {
+                    if (err) {
+                        this.logw(`Error fetching tx note for ${txp.txid}:`, err);
+                    }
+                    txp.note = note;
+                    return cb(null, txp);
+                }, wallet);
+            }, wallet);
         });
     }
     editTxNote(opts, cb) {
@@ -2975,7 +3007,7 @@ class WalletService {
                     }
                     return cb(err, txps);
                 });
-            });
+            }, opts.wallet);
         }
     }
     getPendingTxsPromise(opts) {
@@ -3757,12 +3789,12 @@ class WalletService {
                             this.storage.fetchTxs(this.walletId, {
                                 minTs,
                                 maxTs
-                            }, done);
+                            }, done, wallet);
                         },
                         done => {
                             this.storage.fetchTxNotes(this.walletId, {
                                 minTs
-                            }, done);
+                            }, done, wallet);
                         }
                     ], (err, res) => {
                         return next(err, {