Skip to content

Enhance fee transaction sanity checks.#512

Open
jholdstock wants to merge 1 commit intodecred:masterfrom
jholdstock:fee-tx-inputs
Open

Enhance fee transaction sanity checks.#512
jholdstock wants to merge 1 commit intodecred:masterfrom
jholdstock:fee-tx-inputs

Conversation

@jholdstock
Copy link
Copy Markdown
Member

Confirm the existence of all of the inputs to fee transactions provided by clients. If any of the inputs do not exist, or are not known by the local dcrd instance, an error is returned to the client and the ticket is not registered with the VSP.

Closes #508

Comment thread internal/webapi/payfee.go Outdated
Confirm the existence of all of the inputs to fee transactions provided
by clients. If any of the inputs do not exist, or are not known by the
local dcrd instance, an error is returned to the client and the ticket
is not registered with the VSP.
@jholdstock
Copy link
Copy Markdown
Member Author

I held off on merging this it runs on testnet for a while. I have seen the Fee tx contains non-existent input error 3 times in 16 days, investigating why. Pretty sure its just slow network propagation which I guess is unavoidable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enhance fee transaction sanity checks

2 participants