HomePhabricator

[Cashtab] Patch bug in building token change outputs

Description

[Cashtab] Patch bug in building token change outputs

Summary:
Lingering issue here where users may create a token tx that sends changes to another wallet they control that is not the active wallet.

This is because the constant we are using to manage token change is being mutated by sendXec. So a user sending multiple txs while changing wallets will run into this.

The "right way" to fix this is to have a wallet class that handles utxo mgmt and sends. Working on this. But should patch this now.

Test Plan: npm test

Reviewers: #bitcoin_abc, emack

Reviewed By: #bitcoin_abc, emack

Differential Revision: https://reviews.bitcoinabc.org/D17545

Details

Provenance
bytesofmanAuthored on Thu, Jan 16, 14:09
bytesofmanPushed on Thu, Jan 16, 14:27
Reviewer
Restricted Project
Differential Revision
D17545: [Cashtab] Patch bug in building token change outputs
Parents
rABC0ac291483c70: [mock-chronik-client] Improve path defs for typescript build
Branches
Unknown
Tags
Unknown