unusable message signing (ethereum/accounts/:name/sign and ethereum/accounts/:name/verify)

[immutability-io]

unusable message signing (ethereum/accounts/:name/sign and ethereum/accounts/:name/verify)

[cypherhat]

I will try to take a look at this later this evening.

[mortimr]

Hi @cypherhat, any news on the issue ? Let me know if I can help you with anything

[cypherhat]

I apologize - I have 2 full time commitments... I will definitely try this weekend to evaluate your issue. If you have an implementation suggestion, please suggest. Or submit a PR.

[cypherhat]

I think I should provide a native EIP712 signer method. As I mentioned, I have a heavy workload - I am gonna add this as a feature request. I will make every effort to address this ASAP. Contributions are welcome.

[mortimr]

Hi @cypherhat, I currently have a big task to finish. I’ll look into the issue aswell in ~2-3 days. I think the issue can be solved quite easily by debugging the current “sign” and “verify” method and make sur of their interoperability with other tools. First thing I’ll try is to accept hex encoded arguments for the “sign” method in order to have an encoding format to properly relay binary as hex. I’ll try to debug and compare the results of the keccak256 and ecdsa with other tools and will come back to you then :)

[mortimr]

Hi @cypherhat, the issue came from the keccak256 implementation I was using. The sign route seems to work. But the verify route on the accounts is not working at all. I'll let you look into it.

[mortimr]

Sorry, my bad, the issue came from the hash you are doing without giving the ability to encode. I fixed it locally and will raise a PR with an extra argument encoding on the sign route.

[cypherhat]

Thanks man!

Remember to wash your hands before you submit the PR.

[MonarthS]

@mortimr & @cypherhat any updates on this, as I'm facing same issue.