Ethers.js: Bug: Events with arrays as indexed arguments.

[GFJHogue]

It appears that event parse functions created under Interface.events.SomeEvent() attempt to decode indexed array parameters as their original types.

This is incorrect since the actual event behaviour is to store the keccak256 hash of the array as the topic: http://solidity.readthedocs.io/en/develop/contracts.html#events

This problem leads parse() to attempt to decode bytes32 data as whatever original type it was hashed from. In particular, things crash and burn with indexed dynamic arrays (string, bytes, etc.) when dynamicOffset.value.toNumber() gets called:

Error: Number can only safely store up to 53 bits

[ricmoo]

Thanks for the link @GFJHogue!

I'm going to get a bunch of procedural generated test cases ready this week, so I can make sure all the event API features are working correctly.

[ricmoo]

Fixed in https://github.com/ethers-io/ethers.js/commit/0d106d01af581f31ff77235bccde437dd1c793ea.

Once Travis-CI finishes, I will publish to npm.

[ricmoo]

Please give it a try, and let me know if there are any more problems. :)