-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
creating a tx with more outputs than inputs fails with a misleading error #1236
Comments
We may need to do the output greater than input check (https://github.com/bitpay/bitcore/blob/master/lib/transaction/transaction.js#L222) before the fee is different check? |
That error can probably go before all the other ones. I can have a look when #1226 is merged. |
Yes, that's the same solution we discussed with @eordano On Sat, May 16, 2015, 12:10 dskloet [email protected] wrote:
|
What about missing signatures? Should that check go before fee errors and dust outputs as well? |
I created pull request #1241. |
Uncaught bitcore.ErrorTransactionFeeErrorDifferent: Unspent value is different from specified fee: Unspent value is -33338 but specified fee is 10000 Use Transaction#uncheckedSerialize if you want to skip security checks. See http://bitcore.io/guide/transaction.html#Serialization for more info.
The text was updated successfully, but these errors were encountered: