Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Number can only safely store up to 53 bits during decoding of logs #1971

Closed
chlenc opened this issue Mar 28, 2024 · 5 comments
Closed

Number can only safely store up to 53 bits during decoding of logs #1971

chlenc opened this issue Mar 28, 2024 · 5 comments
Assignees
Labels
awaiting We need further input from the author bug Issue is a bug stale Issue is now stale
Milestone

Comments

@chlenc
Copy link

chlenc commented Mar 28, 2024

Hi guys, we are still playing around the Envio indexer and for now, I found some edge case

I have created a repo that emits event and decodes a logs
https://github.com/compolabs/orderbook-indexer-synchronizer

And with one case I have faced into a error

Number can only safely store up to 53 bits

You can reproduce this

  1. Clone the Repository and install dependencies

    git clone git@github.com:compolabs/orderbook-indexer-synchronizer.git
    cd orderbook-indexer-synchronizer
    npm i
    
  2. Create a .env File
    Add your private key to the .env file.

    echo "ALICE=<YOUR PRIVATE KEY>" >> .env
    
  3. Build Contracts
    Compile the smart contracts.

    forc build
    
  4. Make sure you have this contract in addresses.json

{"contractId":"0xe1631771301d5ba38fb7dc51b2815e9d29ef12cae10c6aa7e492008093cde318","blockNumber":8268037}

  1. Run "should decode data" Test
@danielbate danielbate self-assigned this Apr 17, 2024
@danielbate
Copy link
Contributor

Hey @chlenc, think I'm missing some additional config.

 FAIL  tests/data_encode.test.ts
  ● Test suite failed to run

    DB_NAME is required

       7 |   const result = process.env[name];
       8 |   if (result == null && !optional) {
    >  9 |     throw new Error(`${name} is required`);
         |           ^
      10 |   }
      11 |   return result;
      12 | }

@arboleya arboleya added the bug Issue is a bug label Apr 23, 2024
@danielbate danielbate added the awaiting We need further input from the author label May 6, 2024
@danielbate
Copy link
Contributor

@maschad this doesn't seem to have gone stale yet 🤔

@maschad
Copy link
Member

maschad commented May 20, 2024

@danielbate based on the CI action job it needs to be 14 full days

Copy link
Contributor

github-actions bot commented Jun 4, 2024

Oops, seems like we needed more information for this issue, please comment with more details or this issue will be closed in 7 days.

@github-actions github-actions bot added the stale Issue is now stale label Jun 4, 2024
@arboleya arboleya added the p0 label Jun 9, 2024
@arboleya arboleya added this to the Mainnet milestone Jun 9, 2024
Copy link
Contributor

This issue was closed because it is missing author input.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting We need further input from the author bug Issue is a bug stale Issue is now stale
Projects
None yet
Development

No branches or pull requests

4 participants