catapult-rest icon indicating copy to clipboard operation
catapult-rest copied to clipboard

Published 20 hours ago • verified publisher icon symbol

Consider dropping custom Uint64.js implementation in favour of Node's BigInt

Open Vektrat opened this issue 4 years ago • 3 comments

Vektrat avatar Mar 26 '20 10:03 Vektrat

we will do the same thing on SDK

rg911 avatar Mar 26 '20 10:03 rg911

And the wallet. The wallet is currently converting the SDKs UInt64s to numbers which can overflow

fboucquez avatar Mar 26 '20 10:03 fboucquez

this is an internal change that should not have outside effects (API), we will keep delivering longs in HEX/string format, we are just taking this chance now that we updated node to a version that supports longs

Vektrat avatar Mar 26 '20 13:03 Vektrat