Merge pull request #3203 from ethereum/issue/3175-hotfix

Revert 3123
web3 · Nov 14, 2019 · 72d1da0 · 72d1da0
2 parents 0f64355 + e7fdb98
commit 72d1da0
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -89,6 +89,7 @@ Released with 1.0.0-beta.37 code base.
 
 ### Fixed
 
+- Fix perfect gas usage causes tx to error (#3175)
 - Fix regenerator runtime error in web3.min.js (#3155)
 - Fix TS types for eth.subscribe syncing, newBlockHeaders, pendingTransactions (#3159)
 - Improve web3-eth-abi decodeParameters error message (#3134)
diff --git a/packages/web3-core-method/src/index.js b/packages/web3-core-method/src/index.js
@@ -394,7 +394,7 @@ Method.prototype._confirmTransaction = function(defer, result, payload) {
                 .then(function(receipt) {
                     if (!isContractDeployment && !promiseResolved) {
                         if (!receipt.outOfGas &&
-                            (!gasProvided || gasProvided !== utils.numberToHex(receipt.gasUsed)) &&
+                            (!gasProvided || gasProvided !== receipt.gasUsed) &&
                             (receipt.status === true || receipt.status === '0x1' || typeof receipt.status === 'undefined')) {
                             defer.eventEmitter.emit('receipt', receipt);
                             defer.resolve(receipt);