getblocktemplate does not show SigWit feature in Stratum #13
Labels
No labels
ci
claimtrie
consider soon
Epic
good first issue
hacktoberfest
help wanted
mempool
mining
peer
priority: blocker
priority: high
priority: low
priority: medium
rpc
runtime
test
type: bug
type: discussion
type: improvement
type: new feature
type: refactor
type: task
ux
wallet
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
LBRYCommunity/lbcd#13
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
This might need changes on either
Stratumorlbcdside, such as magic number, coin symbols, etc.Update:
Stratum tries to parse the
rulesfield to find thesegwitsupport, but lbcd only appendssegwittorulesif the blocktemplate includes any segwit transactions.Will update again after consulting BIP 145.
Update:
rulesfield is optional when there's no SegWit transactions (inputs) in the template.default_witness_commitfield is optional when there's no SegWit transactions (inputs) in the template.lbcd does correctly implement the above behaviors - populate both fields when SwgWit inputs are included in the template, and omit both of the fields otherwise. However, the
default_witness_commitreturned is raw commitment instead of the witness script. (see issue #20)This poses 2 questions:
getblocktemplateresponse?The Segwit related fields such as
default_witness_commitmentand rules ["segwit"] is required only when a template includes SegWit transactions.