Update CTransaction::IsStandard() to return a string describing the reason a transaction is considered non-standard. Add that string to the TX-is-nonstd error message.
Log reason for non-standard transaction rejection #2743
pull jgarzik wants to merge 1 commits into bitcoin:master from jgarzik:reject-reason changing 4 files +38 −18-
jgarzik commented at 6:54 PM on June 5, 2013: contributor
-
-
sipa commented at 5:36 PM on June 22, 2013: member
ACK, but needs a rebase.
-
Log reason for non-standard transaction rejection 980bfe6ef8
-
jgarzik commented at 6:06 AM on June 23, 2013: contributor
Rebased.
-
mikehearn commented at 10:17 AM on June 25, 2013: contributor
LGTM
You could check the reason code in the unit tests to ensure the transactions are being considered non-std for the right reasons. But it's a minor thing.
-
mikehearn commented at 10:42 AM on July 5, 2013: contributor
Let's merge this?
- jgarzik referenced this in commit d598872726 on Jul 10, 2013
- jgarzik merged this on Jul 10, 2013
- jgarzik closed this on Jul 10, 2013
- jgarzik deleted the branch on Aug 24, 2014
- Bushstar referenced this in commit b5466e20a9 on Apr 5, 2019
- DrahtBot locked this on Sep 8, 2021
