aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* automatic DAG pregeneration for smooth epoch transitionszelig2015-05-214-1/+127
| | | | | | | | - backend: AutoDAG bool flag passed from cli/eth.Config to ethereum, autoDAG loop started if true - backend: autoDAG loop start/stop, remove previous DAG - cli: AutoDAG bool flag, off by default, but automatically ON if mining - admin jsre: add startAutoDAG stopAutoDAG and makeDAG in miner section - switch on/off DAG autogeneration when miner started/stopped on console
* Merge pull request #1062 from Gustav-Simonsson/tests_updatesJeffrey Wilcke2015-05-217-31/+9209
|\ | | | | Tests updates
| * Add wrapper for BlockTests/bcWalletTest.jsonGustav Simonsson2015-05-181-0/+4
| |
| * Handle call depth exception for CREATEGustav Simonsson2015-05-181-0/+6
| |
| * Update Ethereum JSON test filesGustav Simonsson2015-05-185-31/+9199
| |
* | Merge pull request #1059 from obscuren/cleanupJeffrey Wilcke2015-05-215-36/+37
|\ \ | |/ |/| Cleanup
| * miner: one-shot update loopobscuren2015-05-211-0/+6
| |
| * eth: reduced max open files for LevelDBobscuren2015-05-211-1/+1
| |
| * ethdb: documentation and corruption recoveryobscuren2015-05-211-8/+13
| |
| * core: switched back to `set.Set` for uncle verificationobscuren2015-05-211-24/+14
| |
| * xeth: removed `Value`obscuren2015-05-211-3/+3
|/
* Merge pull request #1043 from obscuren/test_fixesJeffrey Wilcke2015-05-203-13/+35
|\ | | | | core/vm: optimisation on RETURN and updated tests
| * tests: added conditional skip on long running VM testsobscuren2015-05-201-0/+7
| | | | | | | | | | Set the TEST_VM_COMPLEX env var to test complex vm tests which require a lot of ram and quite some time.
| * core/vm: RETURN op code returns pointer to memory rather than copyobscuren2015-05-203-5/+17
| |
| * tests/vm: updated tests and skipped output for specific testsobscuren2015-05-201-8/+11
| | | | | | | | Skipped tests due to large return value
* | Merge pull request #1049 from zsfelfoldi/receiptsJeffrey Wilcke2015-05-202-0/+34
|\ \ | | | | | | Storing tx receipts in extraDb
| * | Storing tx receipts in extraDbzsfelfoldi2015-05-202-0/+34
| | |
* | | Merge pull request #1048 from ethersphere/cli-fixesJeffrey Wilcke2015-05-2015-645/+2305
|\ \ \ | | | | | | | | CLI, JSRE admin and Solc improvements
| * | | rpc: NewNotAvailableError instead of NewNotImplementedError if no solczelig2015-05-201-1/+1
| | | |
| * | | fix eth.sign. now implemented in admin jsre until web3.js has it .zelig2015-05-204-43/+68
| | | |
| * | | multiple contract source for solidity compiler: returns contract array if ↵zelig2015-05-206-74/+91
| | | | | | | | | | | | | | | | multiple contracts. fixes #1023
| * | | update web3.js to 0.4.2zelig2015-05-201-475/+2061
| | | |
| * | | add usage doc to wallet importzelig2015-05-201-1/+9
| | | |
| * | | remove solc flairzelig2015-05-201-1/+2
| | | |
| * | | solc now in ethereum, fixes solc path setting; setSolc() didnt workzelig2015-05-206-15/+26
| | | |
| * | | fix solc tests unskipzelig2015-05-206-38/+50
| |/ /
* | | Merge pull request #1050 from karalabe/handle-slow-download-testerJeffrey Wilcke2015-05-201-1/+1
|\ \ \ | | | | | | | | eth/downloader: fix test to it doesn't time out on a slow machine
| * | | eth/downloader: fix test to it doesn't time out on a slow machinePéter Szilágyi2015-05-201-1/+1
| | | |
* | | | Merge pull request #1051 from karalabe/fix-odd-namingJeffrey Wilcke2015-05-202-7/+7
|\ \ \ \ | |_|/ / |/| | | eth: fix odd method names in peer set
| * | | eth: fix odd method names in peer setPéter Szilágyi2015-05-202-7/+7
|/ / /
* | | Merge pull request #1044 from obscuren/thread_safe_block_cacheJeffrey Wilcke2015-05-201-0/+3
|\ \ \ | | | | | | | | core: block cache Has method thread safe
| * | | core: block cache Has method thread safeobscuren2015-05-201-0/+3
| | |/ | |/|
* | | Merge pull request #1046 from tgerring/issue1045Jeffrey Wilcke2015-05-201-10/+21
|\ \ \ | |/ / |/| | Allow unlocking multiple accounts
| * | Support multiple account unlock attemptsTaylor Gerring2015-05-201-3/+10
| | |
| * | Allow unlocking multiple accounts #1045Taylor Gerring2015-05-201-9/+13
| | | | | | | | | | | | Separate accounts with spaces when using --unlock
* | | core: skipped tests while cache disabledobscuren2015-05-201-0/+2
|/ /
* | Merge pull request #1035 from karalabe/eth-threadsafe-peersJeffrey Wilcke2015-05-193-68/+150
|\ \ | | | | | | eth: make the peer-set thread safe
| * | eth: remote a superfluous peerSet methodPéter Szilágyi2015-05-191-12/+0
| | |
| * | eth: make the peer set thread safePéter Szilágyi2015-05-193-68/+162
| |/
* | Merge pull request #1036 from tgerring/issue884Jeffrey Wilcke2015-05-192-8/+21
|\ \ | | | | | | JSON RPC null field updates
| * | Use bytes.Repeat() instead of 32-byte literalTaylor Gerring2015-05-191-1/+2
| | |
| * | XEth comment clarificationTaylor Gerring2015-05-181-2/+5
| | |
| * | CleanupTaylor Gerring2015-05-181-3/+1
| | |
| * | Return nil for certain fields on eth_getTransactionByHash when not part of a ↵Taylor Gerring2015-05-181-3/+7
| | | | | | | | | | | | block
| * | Return nil for certain fields on eth_getBlockByNumber pendingTaylor Gerring2015-05-181-0/+7
| | |
* | | core: disable cash tmpobscuren2015-05-191-3/+5
| | |
* | | Merge pull request #1034 from obscuren/tx_pool_fixJeffrey Wilcke2015-05-191-0/+2
|\ \ \ | | | | | | | | Nonce fix for failing transactions
| * | | xeth: remove nonce on error. Fixes #1026obscuren2015-05-191-0/+2
| |/ /
* | | Merge pull request #1022 from obscuren/parallel_nonce_checksJeffrey Wilcke2015-05-199-25/+119
|\ \ \ | | | | | | | | Parallelise nonce checks
| * | | core: fixed testsobscuren2015-05-193-8/+15
| | | |
| * | | core: chain manager no longer exports genesis blockobscuren2015-05-192-2/+4
| | | |
| * | | miner: stale block notificationobscuren2015-05-191-1/+7
| | | |
| * | | core: parallelise nonce checking when processing blocksobscuren2015-05-184-15/+94
| | | | | | | | | | | | | | | | | | | | | | | | ChainManager now uses a parallel approach to block processing where all nonces are checked seperatly from the block processing process. This speeds up the process by about 3 times on my i7
* | | | Merge pull request #1033 from tgerring/issue1010Jeffrey Wilcke2015-05-193-33/+70
|\ \ \ \ | |_|_|/ |/| | | Add "removedb" command to Geth
| * | | Remove unused confirm() methodTaylor Gerring2015-05-191-14/+0
| | | |
| * | | Refactor user prompts into utilsTaylor Gerring2015-05-193-49/+50
| | | |
| * | | Add user confirmation for removedbTaylor Gerring2015-05-191-5/+40
| | | |
| * | | Add "removedb" command to GethTaylor Gerring2015-05-181-0/+15
| | | | | | | | | | | | | | | | Removes the state and blockchain databases
* | | | Merge pull request #1032 from tgerring/issue1025Jeffrey Wilcke2015-05-181-1/+2
|\ \ \ \ | |/ / / |/| | | Multiple CORS domains support
| * | | Permit multiple CORS domainsTaylor Gerring2015-05-181-1/+2
|/ / / | | | | | | | | | Separated by spaces
* | | Merge pull request #1031 from bas-vk/issue1012Jeffrey Wilcke2015-05-181-12/+13
|\ \ \ | |_|/ |/| | fixed race condition in miner
| * | fixed issue when miner is not stopping af stop commandBas van Kervel2015-05-181-0/+6
| | |
| * | fixed race condition in minerBas van Kervel2015-05-181-12/+7
|/ /
* | core: bugfix test 2. set => hash mapobscuren2015-05-181-14/+23
| |
* | retobscuren2015-05-181-0/+1
| |
* | core: tmp diagnostic logsobscuren2015-05-181-0/+7
| |
* | Merge branch 'hotfix/0.9.22' into developobscuren2015-05-181-1/+1
|\ \ | |/ |/|
| * cmd/geth: fixed ver numobscuren2015-05-181-1/+1
| |
| * Merge branch 'hotfix/0.9.21-1'obscuren2015-05-165-27/+36
| |\
* | \ Merge pull request #1021 from obscuren/global_chain_lockJeffrey Wilcke2015-05-171-44/+43
|\ \ \ | | | | | | | | core: global chain insert lock
| * | | core: global chain insert lockobscuren2015-05-171-44/+43
| | | |
* | | | Merge pull request #1011 from obscuren/upgradedb_fixJeffrey Wilcke2015-05-161-0/+1
|\ \ \ \ | |/ / / |/| | | cmd/geth: delete state db on upgradedb command
| * | | cmd/geth: delete state db on upgradedb commandobscuren2015-05-161-0/+1
|/ / /
* | | Merge branch 'hotfix/0.9.21.1' into developobscuren2015-05-164-27/+35
|\ \ \ | | |/ | |/|
| * | cmd/geth: hotfix bumpv0.9.21.1obscuren2015-05-161-1/+1
| | |
| * | core: further improved uncle error messagesobscuren2015-05-161-8/+8
| | |
| * | eth/downloader: moved start eventobscuren2015-05-162-2/+3
| | | | | | | | | | | | Start event has moved because it could possibly could stall the miner
| * | Adjust miner coinbase and not just miner worker coinbaseVitalik Buterin2015-05-161-0/+1
| | |
| * | miner: mutex locks on cpu agent. Closes #1007obscuren2015-05-161-16/+23
| |/
| * Merge branch 'release/0.9.21'v0.9.21obscuren2015-05-16200-3242/+611118
| |\
| * \ Merge branch 'release/0.9.20'v0.9.20obscuren2015-05-1354-617/+4653
| |\ \
| * \ \ Merge branch 'develop' of github.com-obscure:ethereum/go-ethereumobscuren2015-05-100-0/+0
| |\ \ \
* | | | | Adjust miner coinbase and not just miner worker coinbaseVitalik Buterin2015-05-161-0/+1
| | | | |
* | | | | Merge branch 'release/0.9.21' into developobscuren2015-05-166-8/+6
|\ \ \ \ \ | | |_|_|/ | |/| | |
| * | | | core, cmd/geth, cmd/mist: cleanup. bump version 0.9.21obscuren2015-05-164-7/+3
| | | | |
| * | | | xeth: set balance max balance to from in call. Closes #951obscuren2015-05-162-1/+3
|/ / / /
* | | | Merge pull request #989 from obscuren/developJeffrey Wilcke2015-05-164-9/+20
|\ \ \ \ | | | | | | | | | | core, miner: fork resolving and restart miner after sync op
| * | | | eth, cmd/geth: start mining from console respects CLI flagobscuren2015-05-162-1/+3
| | | | |
| * | | | miner: properly check for mining operation on Registerobscuren2015-05-161-1/+1
| | | | |
| * | | | core, miner: fork resolving and restart miner after sync opobscuren2015-05-162-7/+16
|/ / / / | | | | | | | | | | | | Fork resolving fixes #940
* | | | Merge pull request #996 from karalabe/fix-potential-crosscheck-raceJeffrey Wilcke2015-05-161-5/+7
|\ \ \ \ | | | | | | | | | | eth/downloader: circumvent download race between crosscheck and hashes
| * | | | eth/downloader: circumvent download race between crosscheck and hashesPéter Szilágyi2015-05-161-5/+7
| | | | |
* | | | | Merge pull request #997 from karalabe/fix-shitty-test-server-issueJeffrey Wilcke2015-05-161-0/+6
|\ \ \ \ \ | |/ / / / |/| | | | eth/downloader: fix #992, where tests may time out on a slow machine
| * | | | eth/downloader: fix #992, where tests may time out on a slow machinePéter Szilágyi2015-05-161-0/+6
|/ / / /
* | | | Merge pull request #995 from ↵Jeffrey Wilcke2015-05-1620-2741/+420841
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | Gustav-Simonsson/update_state_tests_and_vm_create_return Update state tests and vm create return
| * | | | Add StateTests/RandomTests but skip for nowGustav Simonsson2015-05-161-0/+21
| | | | |
| * | | | Update StateTests and nil create return on failed code depositGustav Simonsson2015-05-162-2/+13
| | | | |
| * | | | Update Ethereum JSON testsGustav Simonsson2015-05-1517-2734/+420801
| | | | |
| * | | | unskip StateTests/stTransactionTest and use test tx nonceGustav Simonsson2015-05-152-5/+6
| | | | |
* | | | | Merge pull request #994 from karalabe/fix-cancel-double-closeJeffrey Wilcke2015-05-161-3/+8
|\ \ \ \ \ | | | | | | | | | | | | eth/downloader: fix cancel channel double close
| * | | | | eth/downloader: fix cancel channel double closePéter Szilágyi2015-05-161-3/+8
|/ / / / /
* | | | | Merge pull request #988 from karalabe/fix-downloader-vulnerabilitiesJeffrey Wilcke2015-05-154-122/+336
|\ \ \ \ \ | |/ / / / |/| | | | Fix downloader vulnerabilities
| * | | | eth/downloader: circumvent a fake blockchain attackPéter Szilágyi2015-05-152-27/+58
| | | | |
| * | | | eth/downloader: don't penalize for stale cross checksPéter Szilágyi2015-05-151-7/+17
| | | | |
| * | | | eth, eth/downloader: detect and handle madeup hash attacksPéter Szilágyi2015-05-154-43/+110
| | | | |
| * | | | eth: drop a sync peer if it sends an invalid hash chainPéter Szilágyi2015-05-151-1/+1
| | | | |
| * | | | eth/downloader: circumvent hash reordering attacksPéter Szilágyi2015-05-153-49/+88
| | | | |
| * | | | eth/downloader: check sync after failed attacksPéter Szilágyi2015-05-152-7/+23
| | | | |
| * | | | eth/downloader: add a test for detecting missing blocksPéter Szilágyi2015-05-152-8/+24
| | | | |
| * | | | eth, eth/downloader: prevent hash repeater attackPéter Szilágyi2015-05-154-15/+50
|/ / / /
* | | | Merge pull request #987 from obscuren/developJeffrey Wilcke2015-05-1512-35/+110
|\ \ \ \ | | | | | | | | | | Miner updates and downloader events
| * | | | eth/downloader: fixed testobscuren2015-05-152-2/+4
| | | | |
| * | | | core: fixed mining strategyobscuren2015-05-152-10/+16
| | | | |
| * | | | eth, eth/downloader, miner: use download events to check miner startobscuren2015-05-156-18/+83
| | | | |
| * | | | core: changed how head events are checkedobscuren2015-05-152-2/+4
| | | | |
| * | | | core: improved uncle messagesobscuren2015-05-151-3/+3
| | | | |
* | | | | Merge pull request #984 from tgerring/issue924Jeffrey Wilcke2015-05-151-3/+21
|\ \ \ \ \ | | | | | | | | | | | | Omit replies for RPC notification requests
| * | | | | Unreverse orderingTaylor Gerring2015-05-151-2/+1
| | | | | |
| * | | | | Omit replies for notification requestsTaylor Gerring2015-05-151-3/+22
| | | | | | | | | | | | | | | | | | | | | | | | When Id is missing, the client does not want a response
* | | | | | Merge pull request #975 from obscuren/developJeffrey Wilcke2015-05-151-11/+13
|\ \ \ \ \ \ | | |/ / / / | |/| | | | core: changed log message for forks. closes #952
| * | | | | core: changed log message for forks. closes #952obscuren2015-05-141-11/+13
| | | | | |
* | | | | | Merge pull request #983 from tgerring/issue958Jeffrey Wilcke2015-05-151-0/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | Format contract code as hexdata
| * | | | | | Format contract code as hexdataTaylor Gerring2015-05-151-0/+1
| | |/ / / / | |/| | | |
* | | | | | Merge pull request #982 from tgerring/issue962Jeffrey Wilcke2015-05-151-1/+5
|\ \ \ \ \ \ | | | | | | | | | | | | | | Check for error on DB fetch of tx
| * | | | | | Check for error on DB fetch of txTaylor Gerring2015-05-151-1/+5
| |/ / / / /
* | | | | | Merge pull request #980 from fjl/geth-test-no-scryptJeffrey Wilcke2015-05-151-1/+1
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | cmd/geth: use plain keystore for JS tests
| * | | | | cmd/geth: use plain keystore for JS testsFelix Lange2015-05-141-1/+1
|/ / / / / | | | | | | | | | | | | | | | This reduces the time it takes to run the tests back to 3 seconds.
* | | | | Merge pull request #974 from karalabe/downloader-fix-unknown-parent-attackJeffrey Wilcke2015-05-144-38/+92
|\ \ \ \ \ | | | | | | | | | | | | eth, eth/downloader: handle a potential unknown parent attack
| * | | | | eth/downloader: add a small additional check to the testPéter Szilágyi2015-05-141-0/+3
| | | | | |
| * | | | | eth/downloader: remove a previous leftoverPéter Szilágyi2015-05-141-1/+0
| | | | | |
| * | | | | eth, eth/downloader: remove parent verification from the downlaoderPéter Szilágyi2015-05-144-46/+17
| | | | | |
| * | | | | eth, eth/downloader: handle a potential unknown parent attackPéter Szilágyi2015-05-143-31/+112
| | | | | |
* | | | | | Merge pull request #971 from fjl/p2p-limit-tweaksJeffrey Wilcke2015-05-147-64/+16
|\ \ \ \ \ \ | | | | | | | | | | | | | | p2p: tweak connection limits
| * | | | | | p2p/discover: limit open files for node databaseFelix Lange2015-05-141-2/+3
| | | | | | |
| * | | | | | p2p: remove testlogFelix Lange2015-05-143-51/+0
| | | | | | |
| * | | | | | p2p/discover: bump maxBondingPingPongs to 16Felix Lange2015-05-141-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This should increase the speed a bit because all findnode results (up to 16) can be verified at the same time.
| * | | | | | p2p: log remote reason when disconnect is requestedFelix Lange2015-05-142-8/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The returned reason is currently not used except for the log message. This change makes the log messages a bit more useful. The handshake code also returns the remote reason.
| * | | | | | p2p: bump maxAcceptConns and defaultDialTimoutFelix Lange2015-05-141-2/+2
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | On the test network, we've seen that it becomes harder to connect if the queues are so short.
* | | | | | Merge pull request #973 from bas-vk/issue972Jeffrey Wilcke2015-05-142-0/+10
|\ \ \ \ \ \ | | | | | | | | | | | | | | Introduced default unlock duration when an account is unlocked from t…
| * | | | | | Introduced default unlock duration when an account is unlocked from the consoleBas van Kervel2015-05-142-0/+10
| | | | | | |
* | | | | | | Merge pull request #960 from fjl/nat-fixesJeffrey Wilcke2015-05-14142-25/+189479
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | p2p/nat: fix UPnP auto discovery
| * | | | | | | cmd/utils: print messages from package log through glogFelix Lange2015-05-141-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some of the dependencies use package log. This change ensures that the log output looks uniform and respects glog flags.
| * | | | | | | p2p/nat: tweak port mapping log messages and levelsFelix Lange2015-05-141-7/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | People stil get confused about the messages. This commit changes the levels so that the only thing printed at the default level (info) is a successful mapping.
| * | | | | | | Godeps: bump github.com/huin/goupnp to c57ae84Felix Lange2015-05-14138-1/+189186
| | | | | | | |
| * | | | | | | p2p/nat: add test for UPnP auto discovery via SSDPFelix Lange2015-05-141-0/+223
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The test listens for multicast UDP packets on the default interface because I couldn't get it to work reliably on loopback without massive changes to goupnp. This means that the test might fail when there is a UPnP-enabled router attached on that interface. I checked that locally by looping the test and it passes reliably because the local SSDP server always responds faster.
| * | | | | | | p2p/nat: fix concurrent access to autodisc InterfaceFelix Lange2015-05-142-17/+63
| | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Concurrent calls to Interface methods on autodisc could return a "not discovered" error if the discovery did not finish before the call. autodisc.wait expected the done channel to carry the found Interface but it was closed instead. The fix is to use sync.Once for now, which is easier to get right. And there is a test. Finally. This will have to change again when we introduce re-discovery.
* | | | | | | Merge pull request #969 from Gustav-Simonsson/ethash_improve_hashrate_update2Jeffrey Wilcke2015-05-145-116/+48
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Make read of ethash hashrate atomic and update ethash godep
| * | | | | | | Make read of ethash hashrate atomic and update ethash godepGustav Simonsson2015-05-145-116/+48
| |/ / / / / /
* | | | | | | Merge pull request #950 from karalabe/downloader-cancel-syncJeffrey Wilcke2015-05-141-32/+37
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | eth/downloader: fix deliveries to check for sync cancels
| * | | | | | | eth/downloader: remove a redundant sync progress checkPéter Szilágyi2015-05-131-4/+0
| | | | | | | |
| * | | | | | | eth/downloader: add a user sync notificatonPéter Szilágyi2015-05-131-0/+5
| | | | | | | |
| * | | | | | | eth/downloader: sync the cancel channel during cancel tooPéter Szilágyi2015-05-131-21/+4
| | | | | | | |
| * | | | | | | eth/downloader: fix deliveries to check for sync cancelsPéter Szilágyi2015-05-131-10/+31
| | |_|/ / / / | |/| | | | |
* | | | | | | Merge pull request #949 from bas-vk/issue944Jeffrey Wilcke2015-05-141-1/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | removed invalid filename characters
| * | | | | | | removed invalid filename charactersBas van Kervel2015-05-131-1/+1
| | | | | | | |
* | | | | | | | Merge pull request #970 from ethereum/vitalik_quick_testJeffrey Wilcke2015-05-141-2/+8
|\ \ \ \ \ \ \ \ | |_|_|_|/ / / / |/| | | | | | | Created separate family and ancestors environment objects
| * | | | | | | Combined two loopsVitalik Buterin2015-05-141-3/+1
| | | | | | | |
| * | | | | | | Created separate family and ancestors environment objectsVitalik Buterin2015-05-141-2/+10
|/ / / / / / /
* | | / / / / Fixed max uncle count error messageVitalik Buterin2015-05-141-1/+1
| |_|/ / / / |/| | | | |
* | | | | | Merge pull request #966 from fjl/fixup-discover-chunked-neighborsFelix Lange2015-05-142-32/+49
|\ \ \ \ \ \ | | | | | | | | | | | | | | p2p/discover: fix out-of-bounds slicing for chunked neighbors packets
| * | | | | | p2p/discover: fix out-of-bounds slicing for chunked neighbors packetsFelix Lange2015-05-142-32/+49
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The code assumed that Table.closest always returns at least 13 nodes. This is not true for small tables (e.g. during bootstrap).
* | | | | | | Merge pull request #963 from Gustav-Simonsson/fix_keystore_crypto_commentsJeffrey Wilcke2015-05-141-6/+7
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | Update keystore code comments
| * | | | | | Update keystore code commentsGustav Simonsson2015-05-141-6/+7
| | | | | | |
* | | | | | | Merge pull request #965 from subtly/patch-1Felix Lange2015-05-142-4/+20
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | Better UDP & interop. Limit all received datagrams to 1280bytes.
| * | | | | | fix test.subtly2015-05-141-1/+1
| | | | | | |
| * | | | | | Manual send of multiple neighbours packets. Test receiving multiple ↵subtly2015-05-142-3/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | neighbours packets.
| * | | | | | UDP Interop. Limit datagrams to 1280bytes.subtly2015-05-141-1/+1
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We don't have a UDP which specifies any messages that will be 4KB. Aside from being implemented for months and a necessity for encryption and piggy-backing packets, 1280bytes is ideal, and, means this TODO can be completed! Why 1280 bytes? * It's less than the default MTU for most WAN/LAN networks. That means fewer fragmented datagrams (esp on well-connected networks). * Fragmented datagrams and dropped packets suck and add latency while OS waits for a dropped fragment to never arrive (blocking readLoop()) * Most of our packets are < 1280 bytes. * 1280 bytes is minimum datagram size and MTU for IPv6 -- on IPv6, a datagram < 1280bytes will *never* be fragmented. UDP datagrams are dropped. A lot! And fragmented datagrams are worse. If a datagram has a 30% chance of being dropped, then a fragmented datagram has a 60% chance of being dropped. More importantly, we have signed packets and can't do anything with a packet unless we receive the entire datagram because the signature can't be verified. The same is true when we have encrypted packets. So the solution here to picking an ideal buffer size for receiving datagrams is a number under 1400bytes. And the lower-bound value for IPv6 of 1280 bytes make's it a non-decision. On IPv4 most ISPs and 3g/4g/let networks have an MTU just over 1400 -- and *never* over 1500. Never -- that means packets over 1500 (in reality: ~1450) bytes are fragmented. And probably dropped a lot. Just to prove the point, here are pings sending non-fragmented packets over wifi/ISP, and a second set of pings via cell-phone tethering. It's important to note that, if *any* router between my system and the EC2 node has a lower MTU, the message would not go through: On wifi w/normal ISP: localhost:Debug $ ping -D -s 1450 52.6.250.242 PING 52.6.250.242 (52.6.250.242): 1450 data bytes 1458 bytes from 52.6.250.242: icmp_seq=0 ttl=42 time=104.831 ms 1458 bytes from 52.6.250.242: icmp_seq=1 ttl=42 time=119.004 ms ^C --- 52.6.250.242 ping statistics --- 2 packets transmitted, 2 packets received, 0.0% packet loss round-trip min/avg/max/stddev = 104.831/111.918/119.004/7.087 ms localhost:Debug $ ping -D -s 1480 52.6.250.242 PING 52.6.250.242 (52.6.250.242): 1480 data bytes ping: sendto: Message too long ping: sendto: Message too long Request timeout for icmp_seq 0 ping: sendto: Message too long Request timeout for icmp_seq 1 Tethering to O2: localhost:Debug $ ping -D -s 1480 52.6.250.242 PING 52.6.250.242 (52.6.250.242): 1480 data bytes ping: sendto: Message too long ping: sendto: Message too long Request timeout for icmp_seq 0 ^C --- 52.6.250.242 ping statistics --- 2 packets transmitted, 0 packets received, 100.0% packet loss localhost:Debug $ ping -D -s 1450 52.6.250.242 PING 52.6.250.242 (52.6.250.242): 1450 data bytes 1458 bytes from 52.6.250.242: icmp_seq=0 ttl=42 time=107.844 ms 1458 bytes from 52.6.250.242: icmp_seq=1 ttl=42 time=105.127 ms 1458 bytes from 52.6.250.242: icmp_seq=2 ttl=42 time=120.483 ms 1458 bytes from 52.6.250.242: icmp_seq=3 ttl=42 time=102.136 ms
* | | | | | Merge pull request #954 from karalabe/fix-downloader-nil-panicPéter Szilágyi2015-05-141-1/+1
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | eth/downloader: fix nil panic caused by wrong variable use
| * | | | | eth/downloader: fix nil panic caused by wrong variable usePéter Szilágyi2015-05-131-1/+1
|/ / / / /
* | | | | Merge pull request #948 from karalabe/fix-downlaoder-activepeer-shadowJeffrey Wilcke2015-05-133-24/+16
|\ \ \ \ \ | |/ / / / |/| | | | eth/downloader: fix active peer shadowing, polish func names
| * | | | eth/downloader: fix active peer shadowing, polish func namesPéter Szilágyi2015-05-133-24/+16
|/ / / /
* | | | Merge pull request #946 from Gustav-Simonsson/fix_geth_unlock_accountJeffrey Wilcke2015-05-131-1/+2
|\ \ \ \ | | | | | | | | | | Fix hex conversion in --unlock and log when successful
| * | | | Fix hex conversion in --unlock and log when successfulGustav Simonsson2015-05-131-1/+2
|/ / / /
* | | | Merge branch 'release/0.9.20' into developobscuren2015-05-132-2/+2
|\ \ \ \ | | |_|/ | |/| |
| * | | eth: 100% tx propagationobscuren2015-05-131-1/+1
| | | |
| * | | cmd/geth: bump version to 0.9.20obscuren2015-05-131-1/+1
|/ / /
* | | Merge pull request #631 from Gustav-Simonsson/improve_key_store_cryptoJeffrey Wilcke2015-05-1322-196/+241
|\ \ \ | | | | | | | | Improve key store crypto
| * | | Change default keystore dirGustav Simonsson2015-05-135-6/+6
| | | |
| * | | Fix address type in js testGustav Simonsson2015-05-121-1/+1
| | | |
| * | | fix hex conversion bug in RPC for byte slicesGustav Simonsson2015-05-121-1/+8
| | | |
| * | | Fix natspec test (again x2) typesGustav Simonsson2015-05-121-1/+2
| | | |
| * | | Update key store to new spec but keep address field for nowGustav Simonsson2015-05-127-72/+59
| | | | | | | | | | | | | | | | * Also fix address types post-rebase
| * | | Fix natspec e2e test accounts type (again)Gustav Simonsson2015-05-121-1/+1
| | | |
| * | | Fix HEX hack in xeth accounts()Gustav Simonsson2015-05-121-1/+1
| | | |
| * | | Address pull request comments; key header and hex encodingGustav Simonsson2015-05-123-31/+67
| | | | | | | | | | | | | | | | | | | | | | | | * Remove key header from unencrypted key file format and replace it with a version field * Change encoding of bytes in key files from base64 to hex
| * | | Revert "Add key header to unencrypted key file"Gustav Simonsson2015-05-122-16/+8
| | | | | | | | | | | | | | | | This reverts commit a94d4ba0b53c4558ab838aaed635a2ff66ddfa53.
| * | | Fix common.Address / []byte type conversionsGustav Simonsson2015-05-122-2/+2
| | | |
| * | | Add key header to unencrypted key fileGustav Simonsson2015-05-122-8/+16
| | | |
| * | | Add key header to encrypted keysGustav Simonsson2015-05-122-7/+57
| | | | | | | | | | | | | | | | | | | | | | | | | | | | * Add key header containing key version, kdf and kdf params * Store key header as JSON in the key file * Read in KDF params from key header * Include key header in MAC calculation and MAC verification
| * | | Correct accounts hex in XETH APIGustav Simonsson2015-05-121-1/+1
| | | |
| * | | Use common.Address type for accounts.AddressGustav Simonsson2015-05-1211-64/+69
| | | |
| * | | Improve key store passphrase cryptoGustav Simonsson2015-05-124-23/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | * Change MAC-then-Encrypt to Encrypt-then-MAC * Change AES256 to AES128 * Use first 16 bytes of KDF derived key for AES and remaining 16 for MAC
| * | | Remove the awesome, ever misunderstood entropy mixingGustav Simonsson2015-05-124-64/+15
|/ / /
* | | Merge pull request #933 from bas-vk/issue928Jeffrey Wilcke2015-05-1218-64/+58
|\ \ \ | | | | | | | | replaced path with platform aware filepath module
| * | | fixed identingBas van Kervel2015-05-121-1/+1
| | | |
| * | | removed redundant newlines in import blockBas van Kervel2015-05-127-11/+5
| | | |
| * | | replaced several path.* with filepath.* which is platform independentBas van Kervel2015-05-1218-64/+64
| | | |
* | | | Merge pull request #932 from obscuren/developJeffrey Wilcke2015-05-127-15/+40
|\ \ \ \ | |/ / / |/| | | xeth, rpc: implement eth_estimateGas. Closes #930
| * | | sol: skipped source checking stepobscuren2015-05-123-7/+13
| | | |
| * | | xeth: use same semantics as block processer for transient callsobscuren2015-05-121-4/+3
| | | |
| * | | rpc: hexData => hexNumobscuren2015-05-121-2/+2
| | | |
| * | | xeth, rpc: implement eth_estimateGas. Closes #930obscuren2015-05-123-7/+26
| | | |
| * | | eth, ethdb: lower the amount of open files & improve err messages for dbobscuren2015-05-121-0/+1
|/ / / | | | | | | | | | Closes #880
* | | Merge pull request #899 from ethersphere/solc-test-fixJeffrey Wilcke2015-05-124-6/+21
|\ \ \ | | | | | | | | compiler, cli/js: fix solidity compiler tests
| * | | compiler, cli/js: fix solidity compiler tests failing when a different ↵zelig2015-05-124-6/+21
| | | | | | | | | | | | | | | | version of solc installed
* | | | Merge pull request #929 from obscuren/developJeffrey Wilcke2015-05-122-5/+8
|\ \ \ \ | |/ / / |/| | | eth, ethdb: lower the amount of open files & improve err messages for db
| * | | eth, ethdb: lower the amount of open files & improve err messages for dbobscuren2015-05-122-5/+8
|/ / / | | | | | | | | | Closes #880
* | | Merge pull request #925 from obscuren/worker_owned_accountsJeffrey Wilcke2015-05-122-29/+44
|\ \ \ | | | | | | | | miner, cmd/geth: miner will not ignored owned account transactions
| * | | miner, cmd/geth: miner will not ignored owned account transactionsobscuren2015-05-122-29/+44
|/ / / | | | | | | | | | Miner does not ignore low gas txs from accounts that are owned.
* | | Merge pull request #918 from obscuren/cpu_miner_fixesJeffrey Wilcke2015-05-1211-46/+50
|\ \ \ | | | | | | | | cmd/geth, miner, backend, xeth: Fixed miner threads to be settable
| * | | eth, eth/downloader: moved pending queue error message to debugobscuren2015-05-112-3/+4
| | | |
| * | | cmd/geth, miner, backend, xeth: Fixed miner threads to be settableobscuren2015-05-119-43/+46
| | | | | | | | | | | | | | | | | | | | Miner threads are now settable through the admin interface (closes #897) and specify 0 CPU worker threads when eth_getWork is called (closes #916)
| * | | eth/downloader: use count instead of peers, clearerPéter Szilágyi2015-05-112-4/+4
| | | |
| * | | eth/downloader: revert to demotion, use harsher penaltyPéter Szilágyi2015-05-112-8/+19
| | | |
| * | | eth/downloader: fix #910, thread safe peers & polishesPéter Szilágyi2015-05-114-161/+210
| | | |
* | | | Merge pull request #915 from karalabe/downloader-threading-fixesJeffrey Wilcke2015-05-114-159/+219
|\ \ \ \ | |/ / / |/| | | eth/downloader: fix #910, thread safe peers & polishes
| * | | eth/downloader: use count instead of peers, clearerPéter Szilágyi2015-05-112-4/+4
| | | |
| * | | eth/downloader: revert to demotion, use harsher penaltyPéter Szilágyi2015-05-112-8/+19
| | | |
| * | | eth/downloader: fix #910, thread safe peers & polishesPéter Szilágyi2015-05-114-161/+210
| | | |
* | | | Merge pull request #920 from tgerring/issue917Jeffrey Wilcke2015-05-111-2/+2
|\ \ \ \ | | | | | | | | | | Return 32-byte hashes from GetWork
| * | | | Return 32-byte hashes from GetWorkTaylor Gerring2015-05-111-2/+2
| | | | | | | | | | | | | | | | | | | | Ensures that the Get Work results are a consistent length. Closes #917
* | | | | Merge pull request #914 from ethersphere/developJeffrey Wilcke2015-05-116-15/+3801
|\ \ \ \ \ | |/ / / / |/| | | | Signature on arbitrary data using the private keys of an account
| * | | | Only allow doSign to sign hashes, enforced by using the type common.HashDaniel A. Nagy2015-05-111-4/+4
| | | | |
| * | | | Interactive signature creation refactored into separate doSign function.Daniel A. Nagy2015-05-111-20/+20
| | | | |
| * | | | Merge branch 'develop' of github.com:ethereum/go-ethereum into developDaniel A. Nagy2015-05-1122-103/+372
| |\ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | Conflicts: rpc/jeth.go
* | | | | Merge pull request #908 from obscuren/developJeffrey Wilcke2015-05-119-97/+135
|\ \ \ \ \ | | | | | | | | | | | | core, eth, miner: improved tx removal & fatal error on db sync err
| * | | | | miner: added log message for mining operation. #912obscuren2015-05-111-0/+4
| | | | | |
| * | | | | core: optimise pending transaction processingobscuren2015-05-112-86/+102
| | | | | |
| * | | | | rpc: display error message to stdoutobscuren2015-05-111-0/+2
| | | | | |
| * | | | | core, eth, miner: improved tx removal & fatal error on db sync errobscuren2015-05-116-31/+47
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * core: Added GasPriceChange event * eth: When one of the DB flush methods error a fatal error log message is given. Hopefully this will prevent corrupted databases from occuring. * miner: remove transactions with low gas price. Closes #906, #903
* | | | | Merge pull request #911 from karalabe/whisper-graceful-failJeffrey Wilcke2015-05-114-2/+53
|\ \ \ \ \ | |_|/ / / |/| | | | rpc, xeth: fix #881, gracefully handle offline whisper
| * | | | rpc, xeth: fix #881, gracefully handle offline whisperPéter Szilágyi2015-05-114-2/+53
|/ / / /
* | | | Merge branch 'develop' of github.com-obscure:ethereum/go-ethereum into developobscuren2015-05-100-0/+0
|\ \ \ \ | |_|/ / |/| | / | | |/ | |/|
| * | Merge pull request #894 from obscuren/developJeffrey Wilcke2015-05-101-1/+4
| |\ \ | | | | | | | | ethdb: set open file limit to 128 for all leveldbs
| * \ \ Merge pull request #893 from obscuren/developJeffrey Wilcke2015-05-103-2/+106
| |\ \ \ | | | | | | | | | | eth: added a cancel method for the downloader
* | | | | cmd/geth, cmd/mist: version 0.9.19obscuren2015-05-102-2/+2
| |_|/ / |/| | |
* | | | ethdb: set open file limit to 128 for all leveldbsobscuren2015-05-101-1/+4
| |/ / |/| |
* | | eth: added a cancel method for the downloaderobscuren2015-05-103-2/+106
|/ / | | | | | | | | | | | | | | | | Added a cancel method to the downloader which gracefully shuts down any active syncing process (hash fetching or block downloading) and resets the queue and remove any pending blocks. Issue with the downloader which would stall because of an active ongoing process when an invalid block was found.
* | cmd/mist: version bumpobscuren2015-05-091-1/+1
| |
* | Merge pull request #888 from obscuren/developv0.9.18Jeffrey Wilcke2015-05-098-15/+88
|\ \ | | | | | | miner, flags: Configurable gas price & log flag change
| * | cmd/geth: bump version 0.9.18obscuren2015-05-091-1/+1
| | |
| * | miner, tests: fixed block testobscuren2015-05-092-1/+6
| | |
| * | miner: start/stop mutex locked. Closes #887obscuren2015-05-091-0/+9
| | |
| * | miner: moved gasprice to non-methodobscuren2015-05-091-9/+12
| | |
| * | miner, cmd/geth: settable gas price from flags and consoleobscuren2015-05-093-0/+44
| | | | | | | | | | | | | | | * --gasprice "<num>" flag * admin.miner.setGasPrice( <num> )
| * | cmd/geth, cmd/mist, eth, flags: renamed loglevel to verbosityobscuren2015-05-094-13/+25
|/ /
| * Signature test.Daniel A. Nagy2015-05-091-0/+28
| |
| * Merge branch 'develop' of github.com:ethereum/go-ethereum into developDaniel A. Nagy2015-05-0823-441/+1203
| |\ | |/ |/|
* | version 0.9.17v0.9.17obscuren2015-05-081-1/+1
| |
* | Merge pull request #863 from karalabe/ordered-block-downloadJeffrey Wilcke2015-05-0813-383/+939
|\ \ | | | | | | eth/downloader: prioritize block fetch based on chain position, cap memo...