Skip to content
This repository has been archived by the owner on Apr 23, 2024. It is now read-only.

sdk/horizon: make transaction streaming stream only the escrow accounts #320

Open
leighmcculloch opened this issue Sep 10, 2021 · 0 comments

Comments

@leighmcculloch
Copy link
Contributor

The transaction streaming code in the sdk/horizon package streams all transactions for a network. This was done to ensure that transactions were given to the sdk/state package in order to prevent unexpected things occurring like backwards state transitions or old balances.

The sdk/state package has been improved in the following issues and as a result should be able to handle receiving transactions out-of-order as long as for a single account they are received in order.

The changes in those PRs also make duplicate transactions okay too.

We should revisit the sdk/horizon transaction streaming code in the horizon.Streamer and change it so that it only streams transactions from Horizon for the two escrow accounts.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants