txnotify command line option #6199

pull djpnewton wants to merge 1 commits into bitcoin:master from djpnewton:txnotify changing 3 files +18 −0

djpnewton commented at 8:20 AM on May 28, 2015: contributor

similar to 'walletnotify' this command line option executes a command whenever a transaction enters the mempool
test txnotify command 4be34e3888

in src/init.cpp:None in 4be34e3888

 457 | @@ -457,6 +458,14 @@ static void BlockNotifyCallback(const uint256& hashNewTip)
 458 |      boost::thread t(runCommand, strCmd); // thread runs free
 459 |  }
 460 |  
 461 | +static void TxNotifyCallback(const uint256& txid)
 462 | +{
 463 | +    std::string strCmd = GetArg("-txnotify", "");
 464 | +
 465 | +    boost::replace_all(strCmd, "%s", txid.GetHex());
 466 | +    boost::thread t(runCommand, strCmd); // thread runs free

jonasschnelli commented at 8:25 AM on May 28, 2015:

I think it is a bad idea to start a process for each accepted tx. Why not supporting #6103 and go over sockets?

paveljanik commented at 8:29 AM on May 28, 2015: contributor

NACK, see #5328 and #6072 for details.
djpnewton commented at 9:33 AM on May 28, 2015: contributor

I can see this has been covered before. I will take a look at #6103
djpnewton closed this on May 28, 2015
MarcoFalke locked this on Sep 8, 2021

Contributors

djpnewton

jonasschnelli

paveljanik