ACK

ut ACK

ACK, with one nit: please a debug message when an exception happens. I think completely ignoring it is not a good idea for troubleshooting and such.

I was trying to avoid introducing any risk of causing another exception while writing a debug log message.

Moved the catch inside the if (fDebug), and made it print a debug message.

reviewed-but-untested-but-don't-think-testing-is-necessary-ACK

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/p4903_efad808aaece74a0308a77e35ad35e845edb97f8/ 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.

I can test, have a reproduction already.

@gmaxwell That's great

Tests out fine, I see it log the unparsable reject message. ACK.