aurora-is-near / aurora-relayer

[DEPRECATED] Web3-compatible relayer server for Aurora.
https://aurora-is-near.github.io/aurora-relayer/
Creative Commons Zero v1.0 Universal
26 stars 13 forks source link

Execution Reverted #248

Closed mfornet closed 1 year ago

mfornet commented 2 years ago

One partner is having this issue consistently in the last few days while using the public RPC.

{
    "code": -32603,
    "message": 
    "[ethjs-query] while formatting outputs from RPC '{\"value\":{\"code\":-32603,\"data\":{\"code\":3,\"message\":\"execution reverted\",\"data\":\"0x\"}}}'"
}
spilin commented 2 years ago

@mfornet Are there any additional info? Maybe script that I can use to reproduce this?

mfornet commented 2 years ago

It happens from time to time on saving actions on https://moonflowerfarmers.com/.

end0cr1ne commented 2 years ago

To replicate use: Create a farm here: https://moonflowerfarmers.com/

ABI: [Uploading FarmV5.txt…]() Hosted at: 0xDE707357D10D86aE21373b290eAbBA07360896F6 call the sync public function with this payload:

[
    {
        "action": 0,
        "fruit": "1",
        "landIndex": 1,
        "createdAt": 1647615841
    },
    {
        "action": 0,
        "fruit": "1",
        "landIndex": 2,
        "createdAt": 1647615842
    },
    {
        "action": 0,
        "fruit": "1",
        "landIndex": 3,
        "createdAt": 1647615843
    },
    {
        "action": 1,
        "fruit": "0",
        "landIndex": 1,
        "createdAt": 1647615919
    },
    {
        "action": 1,
        "fruit": "0",
        "landIndex": 2,
        "createdAt": 1647615920
    },
    {
        "action": 1,
        "fruit": "0",
        "landIndex": 3,
        "createdAt": 1647615921
    },
    {
        "action": 0,
        "fruit": "1",
        "landIndex": 1,
        "createdAt": 1647615923
    },
    {
        "action": 0,
        "fruit": "1",
        "landIndex": 2,
        "createdAt": 1647615923
    },
    {
        "action": 0,
        "fruit": "1",
        "landIndex": 3,
        "createdAt": 1647615924
    },
    {
        "action": 0,
        "fruit": "1",
        "landIndex": 0,
        "createdAt": 1647615926
    },
    {
        "action": 0,
        "fruit": "1",
        "landIndex": 4,
        "createdAt": 1647615927
    },
    {
        "action": 1,
        "fruit": "0",
        "landIndex": 0,
        "createdAt": 1647615990
    },
    {
        "action": 1,
        "fruit": "0",
        "landIndex": 2,
        "createdAt": 1647615991
    },
    {
        "action": 1,
        "fruit": "0",
        "landIndex": 4,
        "createdAt": 1647615992
    },
    {
        "action": 1,
        "fruit": "0",
        "landIndex": 3,
        "createdAt": 1647615995
    },
    {
        "action": 1,
        "fruit": "0",
        "landIndex": 1,
        "createdAt": 1647615997
    },
    {
        "action": 0,
        "fruit": "1",
        "landIndex": 0,
        "createdAt": 1647615998
    },
    {
        "action": 0,
        "fruit": "1",
        "landIndex": 2,
        "createdAt": 1647615998
    },
    {
        "action": 0,
        "fruit": "1",
        "landIndex": 3,
        "createdAt": 1647615999
    },
    {
        "action": 0,
        "fruit": "1",
        "landIndex": 4,
        "createdAt": 1647615999
    },
    {
        "action": 0,
        "fruit": "1",
        "landIndex": 1,
        "createdAt": 1647616000
    }
]

You can use: Date.now().toString().substr(0,10) to update the timestamps

chad-mcdonald commented 2 years ago

Having the same issue with one of our functions on testnet. To reproduce:

  1. Contract
  2. In the claim() function fill in these values:
  1. Claim and sign the TX with metamask. Which will return this error:

[ethjs-query] while formatting outputs from RPC '{"value":{"code":-32603,"data":{"code":3,"message":"execution reverted","data":"0x8a585be2"}}}'

end0cr1ne commented 2 years ago

@mfornet Any updates here :)

mfornet commented 2 years ago

It seems to me that there is a problem with the contract, and it is being reverted due to some issue inside the logic of the contract. I kind of confirmed this by looking into @chad-mcdonald provided contract, you can see how most of the transactions are failures, and that is not related to the RPC.

However, the RPC should be able to return back a better error message which could be parsed by off-chain tools. This was addressed at https://github.com/aurora-is-near/aurora-relayer/pull/97, but it looks like it wasn't completely addressed at that point. cc @spilin

spilin commented 2 years ago

Hey, @mfornet @end0cr1ne . We will investigate it this week. As to returning proper errors, as of now it's not all of the messages can be returned.

end0cr1ne commented 2 years ago

The only reason I assumed that it was engine related is that these started last week randomly without any change from my end :(

end0cr1ne commented 2 years ago

Hey @spilin did you get a chance to look into this. My app's significantly affected 😔

birchmd commented 2 years ago

The only reason I assumed that it was engine related is that these started last week randomly without any change from my end :(

@end0cr1ne this suddenly started happening on testnet or mainnet? There have been no changes to the Engine on mainnet since Feb 23 (1 month ago).

spilin commented 1 year ago

This repository is deprecated. If issue still persists, please reopen it on a new repository of Aurora Relayer v2.