-
Notifications
You must be signed in to change notification settings - Fork 20.5k
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
Improve error messages #475
Comments
Contracts are expensive. |
For a full list of gas (changes) see https://github.com/ethereum/go-ethereum/blob/develop/vm/common.go#L36 |
Another error message that is unclear:
|
The time stamp issue is because the difficulty is very low for the current test net. It means you mined 2 blocks within a second (the protocol doesn't allow this). Regarding the nonce error, i'll have a look in to that. |
|
swarm/pss: Remove redundant run func param to addpeer
* Reduce istanbul default timeout, cap exp backoff * Ensure round 0 timeout factors in block period
* Log on ValidatorElections * merge master (ethereum#496) * Check message address against signature (ethereum#477) * Check signing validator's address matches msg address * Add comments about use of sig data in tests * Try fix Circle build test failures * Try fix Circle build test failures, take 2 * tx price heap fix (ethereum#471) * contract_comm/currency/currency.go * fixed the txn price-sorted min-heap * merge master (ethereum#490) * Add precompiles to access validator set (ethereum#441) * set max gas to double of the charged gas for the 'intrinsic' smart contract calls (ethereum#472) * set max gas to double of the charged gas for the 'intrinsic' evm operations * addressed PR comments * addressed pr comment * Adds Prepared Certificates to ensure Istanbul liveness (ethereum#366) * Check message address against signature (ethereum#477) * Check signing validator's address matches msg address * Add comments about use of sig data in tests * Try fix Circle build test failures * Try fix Circle build test failures, take 2 * added new option --use-in-memory-discovery-table (ethereum#479) * added new option --use-in-memory-discovery-table * merge master (ethereum#489) * Adds Prepared Certificates to ensure Istanbul liveness (ethereum#366) * Check message address against signature (ethereum#477) * Check signing validator's address matches msg address * Add comments about use of sig data in tests * Try fix Circle build test failures * Try fix Circle build test failures, take 2 * Allow v4/v5 on a bootnode simultaneously, allow mobile to use discv5 (ethereum#454) * changes for isolating celo networks * changes to get unit tests working * changes to add salt in the discovery packets * removed checking for the ip address when handling a reply * added ping-ip-from-packet option to bootnode * for handling expected replies, don't filter on expected sender ip address if --pingIPFromPacket is used * Add v4 flag * Add unhandled and quicken docker builds * Add salt & logs * Add v4 flag * Add PeerDiscovery to mobile node config * Remove logs * Remove hardcoded bootnodes * Add salt & turn on discv5 * Delete hardcoded eth bootnodes * Make Discoveryv5 configurable * Lint * Add comment to bootnode v4/v5 handling * Change PeerDiscovery -> NoDiscovery * Remove mobile geth no discovery * Reduce istanbul default timeout, cap exp backoff (ethereum#475) * Reduce istanbul default timeout, cap exp backoff * Ensure round 0 timeout factors in block period * Sanitize logs (ethereum#495) * Change registry lookup and infrastructure lookup error logs to debug level * Sanitize logs regarding registry deployment * Change empty abi logging and comment * Lower log level from error to warning for potentially outdated istanbul messages * Change back to an error message * Add input length checks for precompiled contracts (ethereum#476) * add input length checks * check exact input length. add a new error for input length. check input in a few more places * add tests that verify the input-length checks for contracts that don't require an evm instance * fix formatting * add comments to explain input length checks * run the formatter * e2e transfer test was failing because it passes in a transaction options object, making the input larger than 96 bytes * e2e tests have revealed that our precompiled contracts need to be tolerant of inputs that are longer than the bytes that are actually read
Add 'bor' user during package installation
* Fix row estimation. * Update libzkp. * more * prepare * finish * upgrade * bump version * fix tests * Update to scroll-prover `v0.7.2`. * fix tests * Update miner/worker.go Co-authored-by: Péter Garamvölgyi <[email protected]> * Update miner/worker.go Co-authored-by: Péter Garamvölgyi <[email protected]> * Reset ccc when skips first tx. * do not unnecessarily skip L1 message * fix ccc reset and improve code readability * seal block on circuitcapacitychecker.ErrUnknown --------- Co-authored-by: HAOYUatHZ <[email protected]> Co-authored-by: Péter Garamvölgyi <[email protected]>
* feat: store skipped txs in local db * bump version * bump version * fix test * include L2 txs in skipped index * goimports * rename more * bump version * fix missed renames * fix the bug when calculating l2TxCount. (ethereum#479) * fix bug when calculate l2 tx count * Update version * fix: exclude L1 message from block payload size validation (ethereum#476) * fix: exclude L1 message from block payload size validation * fix the bug when calculating l2TxCount. (ethereum#479) * fix bug when calculate l2 tx count * Update version * bump version --------- Co-authored-by: maskpp <[email protected]> * fix: update row estimation with scroll-prover `v0.7.2` (ethereum#475) * Fix row estimation. * Update libzkp. * more * prepare * finish * upgrade * bump version * fix tests * Update to scroll-prover `v0.7.2`. * fix tests * Update miner/worker.go Co-authored-by: Péter Garamvölgyi <[email protected]> * Update miner/worker.go Co-authored-by: Péter Garamvölgyi <[email protected]> * Reset ccc when skips first tx. * do not unnecessarily skip L1 message * fix ccc reset and improve code readability * seal block on circuitcapacitychecker.ErrUnknown --------- Co-authored-by: HAOYUatHZ <[email protected]> Co-authored-by: Péter Garamvölgyi <[email protected]> * refactor: simplify ccc revert to snapshot (ethereum#480) * simplify ccc revert to snapshot * Update version.go --------- Co-authored-by: HAOYUatHZ <[email protected]> * feat: use --gcmode=archive and --cache.noprefetch=true by default (ethereum#482) * feat: use --gcmode=archive and --cache.noprefetch=true by default * refuse to start with invalid config * lint * lint * feat(sdk): support compressed response (ethereum#469) * enable use compression algorithm * fix ci * Just enable decode compressed content at ethclient * fix comments --------- Co-authored-by: Haichen Shen <[email protected]> * fix: disable pruning and prefetch if not flags are provided (ethereum#483) Co-authored-by: colin <[email protected]> * fix: update libzkp to use scroll-prover `v0.7.5` (ethereum#484) * Update libzkp to use scroll-prover `v0.7.5`. * Update version. * address comments * bump version * nit --------- Co-authored-by: maskpp <[email protected]> Co-authored-by: Steven <[email protected]> Co-authored-by: HAOYUatHZ <[email protected]> Co-authored-by: iczc <[email protected]> Co-authored-by: HAOYUatHZ <[email protected]> Co-authored-by: Haichen Shen <[email protected]> Co-authored-by: colin <[email protected]>
with
develop
80592f2creating trivial contract https://github.com/jorisbontje/sleth/blob/master/contracts/simple.se with hex code
336000556001600155604f8060146000396063567c01000000000000000000000000000000000000000000000000000000006000350463119fbbd48114156036576001600154016001555b63ea66cf6d811415604d5760015460405260206040f35b505b6000f3
via rpc:
result:
ethereum started with:
now in the
ethereum
logs:Why would it run out of gas, since I am providing
0xc350
= 50000 gas upon creation?Edit: turns out that this 'required' amount of 20000 doesn't apply to the complete TX, but a specific action part of it. The error message can be clarified to reflect this.
The text was updated successfully, but these errors were encountered: