Added a command line option "-txnotify=<cmd>" to set a program called when a (unconfirmed) transaction is received. Works similar to "-blocknotify".
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
-
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
Contributors