Display unknown commands received. #4425

pull rebroad wants to merge 1 commits into bitcoin:master from rebroad:DisplayUnknown changing 1 files +1 −0

rebroad commented at 4:25 AM on June 27, 2014: contributor

For extensibility.
kazcw commented at 4:41 AM on June 27, 2014: contributor

You might want to escape the quotes.
Diapolo commented at 7:29 AM on June 27, 2014: none

Right, you need to escape them properly... idea is good IMHO.
laanwj commented at 7:34 AM on June 27, 2014: member

And please sanitize the string using SanitizeString()
Display unknown commands received. 6ecf3edfee
rebroad commented at 3:08 AM on June 28, 2014: contributor

escaped and sanitized..
BitcoinPullTester commented at 3:40 AM on June 28, 2014: none

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/p4425_6ecf3edfee13f47a920ea6ea7b42b96d0bbbd5fd/ for binaries and test log. This test script verifies pulls every time they are updated. It, however, dies sometimes and fails to test properly. If you are waiting on a test, please check timestamps to verify that the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/ Contact BlueMatt on freenode if something looks broken.
laanwj merged this on Jun 28, 2014
laanwj closed this on Jun 28, 2014
laanwj referenced this in commit c8b6c0a2cf on Jun 28, 2014
MarcoFalke locked this on Sep 8, 2021

Contributors