Notification for transactions #5328

pull jgraef wants to merge 4 commits into bitcoin:master from jgraef:master changing 2 files +9 −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”.
notifications for new transactions 7f912f2161
merged upstream changes 02082bb902
forgot to merge something 01c8d2d0e9
fixed usage 7b593cd4f6
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.
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.
laanwj closed this on Nov 21, 2014
DrahtBot locked this on Sep 8, 2021