Skip to content
This repository has been archived by the owner on Mar 13, 2023. It is now read-only.

Dropped tx while transfer successully #563

Closed
boundless-forest opened this issue Mar 24, 2021 · 4 comments · Fixed by #571
Closed

Dropped tx while transfer successully #563

boundless-forest opened this issue Mar 24, 2021 · 4 comments · Fixed by #571
Assignees
Labels
C-EVM [Component] Something about EVM

Comments

@boundless-forest
Copy link
Member

The Metamask shows that the transaction was dropped, but the transfer was executed successfully

image

@boundless-forest boundless-forest self-assigned this Mar 24, 2021
@boundless-forest boundless-forest added C-EVM [Component] Something about EVM U-Question [Uncategorized] Further information is requested labels Mar 24, 2021
@boundless-forest
Copy link
Member Author

curl --location --request POST 'https://pangolin-rpc.darwinia.network/' \
--header 'Content-Type: application/json' \
--data-raw '{
	"jsonrpc":"2.0",
	"method":"eth_getTransactionByHash",
	"params":[
		"0x914d7f5baddc04e5343024146b3a9d28e435070679892e8bf5ab93cfc584f2a1"
	],
	"id":1
}'

The output:

{“jsonrpc”:“2.0",“result”:null,“id”:1}

@boundless-forest
Copy link
Member Author

From Metamask:

 /**
    checks to see if if the tx's nonce has been used by another transaction
    @param txMeta {Object} - txMeta object
    @emits tx:dropped
    @returns {boolean}
  */
  async _checkIftxWasDropped (txMeta) {
    const { txParams: { nonce, from }, hash } = txMeta
    const nextNonce = await this.query.getTransactionCount(from)
    const { blockNumber } = await this.query.getTransactionByHash(hash) || {}
    if (!blockNumber && parseInt(nextNonce) > parseInt(nonce)) {
        return true
    }
    return false
  }

@boundless-forest boundless-forest removed the U-Question [Uncategorized] Further information is requested label Mar 24, 2021
@boundless-forest
Copy link
Member Author

The pangolin network version 210 has this issue. This issue can be fixed by #559 and wait for version 211 afterward. Close it now.

@boundless-forest
Copy link
Member Author

image

Testing based on pangolin 211, I send five transactions in a row without interruption, and the fourth metamask shows drop, but the transfer is successful.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
C-EVM [Component] Something about EVM
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant