Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

Add nonce tracking #66

Merged
merged 3 commits into from
Sep 6, 2018
Merged

Add nonce tracking #66

merged 3 commits into from
Sep 6, 2018

Conversation

cgewecke
Copy link
Contributor

@cgewecke cgewecke commented Aug 29, 2018

#65
truffle 1079
truffle 763

Addresses:
nonce too low
replacement transaction underpriced

These occur frequently using Infura / load balanced nets. Detailed reproduction and explanation in #65.

[Postscript]
All credit for this goes to Zeppelin engineer @spalladino who diagnosed the problem and mapped out some strategies to address it. In the end it looks like we only have a partial solution here. . . it may fix truffle 763 but truffle 1079 (firing a series of simultaneous txs) will still be a problem. #65 has additional info and a work-around that Zeppelin is using to do this successfully.

@cgewecke cgewecke requested a review from gnidan September 5, 2018 18:48
@@ -18,7 +18,7 @@ You can use this provider wherever a Web3 provider is needed, not just in Truffl

```javascript
var HDWalletProvider = require("truffle-hdwallet-provider");
var mnemonic = "opinion destroy betray ..."; // 12 word mnemonic
var mnemonic = "mountains supernatural bird..."; // 12 word mnemonic
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

giphy

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants