HomePhabricator

[ecash-herald] Patch error in parsing burn txs

Description

[ecash-herald] Patch error in parsing burn txs

Summary:
T2972

Ran into this error running some tests. This tx threw an error because it had tokenBurnInfo but no associated tokenId.

Discussed with Tobias and I've made some incorrect assumptions about how chronik indexes token burns. Will correct this in a later diff. This just patches a potential edge case and makes the app more robust to errors.

Test Plan:
npm test
npm run sendMsgByBlock 798201 and no error

Reviewers: #bitcoin_abc, PiRK

Reviewed By: #bitcoin_abc, PiRK

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

Details

Provenance
bytesofmanAuthored on Jun 28 2023, 23:38
bytesofmanPushed on Jun 29 2023, 13:01
Reviewer
Restricted Project
Differential Revision
D14158: [ecash-herald] Patch error in parsing burn txs
Parents
rABC1bdc3d2d7f1a: [test] Prevent feature_block spurious timeout
Branches
Unknown
Tags
Unknown