You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We hit this bug while trying to use WalletConnect with Dodo.
Test case:
it("should handle number inside string literal",()=>{constnested='{"params":{"proposer":{"metadata":{"description":"Trade Any Token on DODOEX. Swap ETH to WETH at 0.99852536006139370845107244063040676283327993685155310925333096461126073315184832, 0xEeeeeEeeeEeEeeEeEeEeeEEEeeeeEeeeeeeeEEeE, 0x82aF49447D8a07e3bd95BD0d56f35241523fBab1"}}}}';constresult=safeJsonParse(nested);chai.expect(result).to.deep.eq(JSON.parse(nested));});
We hit this bug while trying to use WalletConnect with Dodo.
Test case:
The problem is that safe-json uses a regex to parse big numbers, but this regex does not respect the structure of JSON objects.
It would also be good if the library didn't swallow the error as this causes problems higher-up which are difficult to diagnose.
The text was updated successfully, but these errors were encountered: