Notification for transactions #5328

pull jgraef wants to merge 4 commits into bitcoin:master from jgraef:master changing 2 files +9 −1
  1. jgraef commented at 3:08 pm on November 20, 2014: none
    Added a command line option “-txnotify=” to set a program called when a (unconfirmed) transaction is received. Works similar to “-blocknotify”.
  2. notifications for new transactions 7f912f2161
  3. merged upstream changes 02082bb902
  4. forgot to merge something 01c8d2d0e9
  5. fixed usage 7b593cd4f6
  6. gavinandresen commented at 3:20 pm on November 20, 2014: contributor

    Forking a new process for every single transaction is almost certainly a bad idea.

    See #5303 for a better approach.

  7. laanwj commented at 3:31 pm on November 20, 2014: member

    Oh I was about to comment that we already have -walletnotify=<cmd>, but this is for literally every transaction that another node sends.

    Agree @gavinandresen zmq is the way forward here. Forking is a suboptimal notification mechanism in the best of times, but for something so high-frequency it’s even worse.

  8. laanwj closed this on Nov 21, 2014

  9. DrahtBot locked this on Sep 8, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-11-17 21:12 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me