Page MenuHomePhabricator

[Cashtab] Patch bug in building token change outputs
ClosedPublic

Authored by bytesofman on Thu, Jan 16, 14:10.

Details

Reviewers
emack
Group Reviewers
Restricted Project
Commits
rABC467afd4d59d5: [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

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable