Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace network broadcast API #490

Closed
theoreticalbts opened this issue Oct 5, 2016 · 2 comments
Closed

Replace network broadcast API #490

theoreticalbts opened this issue Oct 5, 2016 · 2 comments

Comments

@theoreticalbts
Copy link
Contributor

We've decided to replace the transaction notification behavior with a polling API, see #486. We want to replace this API while continuing to support clients that relied on the now-deprecated transaction notification mechanism.

So what we're going to do is the following:

  • Don't register any API's by default, require registering network_broadcast_api in the config file
  • Create new network_broadcast2_api which is totally stateless and doesn't do any deferred processing
  • Create transaction state polling method on database_api
  • Update wallet to use network_broadcast2_api
  • Update wallet to use polling
@theoreticalbts
Copy link
Contributor Author

Transaction polling

@theoreticalbts
Copy link
Contributor Author

No code has been written, and this implementation strategy should be revisited. Closing, any further discussion should take place on #486.

@theoreticalbts theoreticalbts removed their assignment Nov 27, 2017
On1x pushed a commit to VIZ-Blockchain/viz-cpp-node that referenced this issue May 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant