Metamask-extension: Error reporting

Created on 31 May 2018 · 7Comments · Source: MetaMask/metamask-extension

I see that you updated error reporting and now instead of printing whole stack trace in the error message you print actually meaningful message, that is nice. Unfortunately it seems that 'error' event is not being fired anymore, instead only the error from MetaMask extension is being printed in the console.error, so as a result, my logic in the .on('error', ...) is ignored.

web3.eth.sendTransaction(transaction)
    .on('transactionHash', resolve)
    .on('error', console.log);

Source

arthurlataks

👍6

Most helpful comment

Until the patch is released, you can install the build from file https://github.com/MetaMask/metamask-extension/pull/4433#issuecomment-393945838

wjmelements on 1 Jun 2018

👍2

All 7 comments

Same problem here. My web3.eth.sendTransaction(transaction) catch is ignored.

fradias on 31 May 2018

Same issue here. Any chance we can get that error event added back in after the new console error so dapps can react to the transaction reject

DaneWiid1 on 1 Jun 2018

This problem is super annoying. Our dApp is broken because of this :cry:

jobsamuel on 1 Jun 2018

@jobsamuel Same here, project due tomorrow ..

ehoelzl on 1 Jun 2018

The builds at https://github.com/MetaMask/metamask-extension/pull/4433#issuecomment-393945838 should fix this bug, feel free to test them until they're released. We plan to merge this fix and to cut a patch release today. Sorry for the issue, we're on it!

bitpshr on 1 Jun 2018

🎉1

can you please let me know when and where can we use the fixed version? because I just installed from chrome store and still seeing the issue.
thanks for your updates!