Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Ethash "sealFields" should return RLP decoded values #6123

Closed
keorn opened this issue Jul 21, 2017 · 3 comments
Closed

Ethash "sealFields" should return RLP decoded values #6123

keorn opened this issue Jul 21, 2017 · 3 comments
Labels
F8-enhancement 🎊 An additional feature request. M4-core ⛓ Core client code / Rust. P7-nicetohave 🐕 Issue is worth doing eventually. Q2-easy 💃 Can be fixed by copy and pasting from StackOverflow.
Milestone
1.10

Comments

@keorn
Copy link

keorn commented Jul 21, 2017

Values in "sealFields" should be equal to "mixHash" and "nonce" fields.
Before someone looks at poor ethereum/EIPs#95
@keorn keorn added M4-core ⛓ Core client code / Rust. Q2-easy 💃 Can be fixed by copy and pasting from StackOverflow. labels Jul 21, 2017
@feynmanliang
Copy link
Contributor

I'd like to work on this. Is the related code under rpc/src/v1/{types,impl}?

@keorn
Copy link
Author

keorn commented Jul 24, 2017

Yep and ethcore/src/ethereum/ethash.rs in fn extra_info

@5chdn 5chdn added P7-nicetohave 🐕 Issue is worth doing eventually. F8-enhancement 🎊 An additional feature request. and removed P7-nicetohave 🐕 Issue is worth doing eventually. labels Sep 5, 2017
@5chdn 5chdn added this to the 1.10 & ... milestone Oct 5, 2017
@tomusdrw
Copy link
Collaborator

Duplicate of #1090

@tomusdrw tomusdrw marked this as a duplicate of #1090 Oct 30, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
F8-enhancement 🎊 An additional feature request. M4-core ⛓ Core client code / Rust. P7-nicetohave 🐕 Issue is worth doing eventually. Q2-easy 💃 Can be fixed by copy and pasting from StackOverflow.
Projects
None yet
Milestone
1.10
Development

No branches or pull requests
4 participants
@tomusdrw @feynmanliang @keorn @5chdn