Skip to content
This repository has been archived by the owner on Aug 11, 2021. It is now read-only.

Decode blocks correctly #14

Closed
vmx opened this issue Jun 27, 2018 · 1 comment
Closed

Decode blocks correctly #14

vmx opened this issue Jun 27, 2018 · 1 comment
Labels
exp/expert Having worked on the specific codebase is important

Comments

@vmx
Copy link
Member

vmx commented Jun 27, 2018

Currently if you pass in a Zcash block that includes transactions, it will only use the header part to calculate the CID. This is wrong.

The correct behaviour (to match the Go IPLD Zcash implementation is to distinguish between zcash-block (multicodec 0xc0), which is the header only and a zcash-tx (multicodec 0xc1).

So building the CID of a zcash-block would then be (as expected by IPLD) to hash the full data, which will then be correct as it only contains the header.

@vmx vmx added help wanted Seeking public contribution on this issue status/ready Ready to be worked exp/expert Having worked on the specific codebase is important labels Jun 27, 2018
@vmx vmx mentioned this issue Jun 27, 2018
@vmx vmx removed the help wanted Seeking public contribution on this issue label Jul 17, 2018
@vmx
Copy link
Member Author

vmx commented Jul 17, 2018

I close this issue in favour of the much clearer one #19.

@vmx vmx closed this as completed Jul 17, 2018
@alanshaw alanshaw removed the status/ready Ready to be worked label Jul 17, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
exp/expert Having worked on the specific codebase is important
Projects
None yet
Development

No branches or pull requests

2 participants