crowetic/protocol
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Update subprovider to catch correct RPC method

Browse Source 
ZeroEx uses web3.eth.sign, rather than web3.eth.personal.sign. In addition, the message to sign is the second parameter; not the first. The first parameter is the address.
This commit is contained in:
Cavan Flynn
2017-12-27 19:48:58 -05:00
committed by GitHub
parent 69806c8839
commit 6ea8cee551
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
packages/subproviders/src/subproviders/ledger.ts
View File

@@ -106,8 +106,8 @@ export class LedgerSubprovider extends Subprovider {
} }
return; return;
case 'personal_sign': case 'eth_sign':
const data = payload.params[0]; const data = payload.params[1];
try { try {
if (_.isUndefined(data)) { if (_.isUndefined(data)) {
throw new Error(LedgerSubproviderErrors.DataMissingForSignPersonalMessage); throw new Error(LedgerSubproviderErrors.DataMissingForSignPersonalMessage);