-
Notifications
You must be signed in to change notification settings - Fork 112
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
web3 server-side changes (JSON RPC spec compliance + bug-fixes) #75
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
need review from either Chris or Ross if this change breaks miner compatibility
@@ -169,8 +169,7 @@ private static JSONObject process(final IRpc.Method _method, final long _id, fin | |||
|
|||
case eth_getWork: | |||
// Header without nonce and solution , pool needs add new nonce | |||
return processResult(_id, api.getBestBlock().getHeader().getHeaderBytes(true)); | |||
|
|||
return processResult(_id, toHexString(HashUtil.h256(api.getBestBlock().getHeader().getHeaderBytes(true)))); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
need review from @aion-6368726973 or @aion-Ross if this will break miner compatibility.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
At this time it should not break the reference miners/pool as they are relying on the "getblocktemplate" call.
However why are we returning a hash of the block header? If we want to update this to return Equihash work we would need eth_getWork to return the entire block header. Our result will be quite different from the referenced eth_getWork.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes shouldn't cause any problems with miner.
courtesy of Shidokht.
fix for #74