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

fromRpcSig and toRpcSig should have an option to include the geth header #63

Closed
axic opened this issue Dec 6, 2016 · 3 comments
Closed

Comments

@axic
Copy link
Member

axic commented Dec 6, 2016

"\x19Ethereum Signed Message:\n" + len(message).

For more see ethereum/go-ethereum#2940.

(From #60.)

@Neurone
Copy link
Contributor

Neurone commented Jul 5, 2018

I think this issue can be closed: since the merge of #67 it seems to me there's no more reason to modify fromRpcSig and toRpcSig.

@holgerd77
Copy link
Member

@Neurone Thanks for taking on on this. Actually I am not getting this issue in the first place, this added Geth header doesn't touch the r,s,v values, or does it?

@Neurone
Copy link
Contributor

Neurone commented Jul 6, 2018

@holgerd77 No, it does not. The Geth header is added to the payload before signing it: hashPersonalMessage already does exactly that.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants