-
Notifications
You must be signed in to change notification settings - Fork 20.1k
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
Fatal: Unlock account failed 'read ~/Library/Ethereum/keys: is a directory' #472
Comments
The address should be supplied to unlock without the leading 0x in this case. @obscuren Can I add a little check to see wether 0x is given or not so both formats work? |
+1 to additionally supporting "0x" format. There are some scattered "fromHex" convenience methods around that could be duplicated |
But of course |
Hi, so after I do this correctly (AFAIK) I'm still getting an error in ethereum.js when I connect through the rpc of "could not unlock sender account"... any suggestions on what I'm doing wrong or what I'm missing? Do I need to supply the password somehow from the ethereum.js side? |
@drcode start ethereum with unlocked account like so: |
Hi @obscuren, I've already been following those steps and still get "could not unlock sender account" from the javascript side. However, it seems to be working fine for others and I'm changing focus right now to UI work anyways, so I'll just check back in a couple of weeks and try again- Thanks for all your great work on this awesome program! |
That's odd. Are you sure you're on the latest |
i'm getting the same problem as drcode. using the devel (0.9.1) version via homebrew |
@scottzer0, make sure your javascript sendTransaction functions all have a "from" field. I think you're running an older build of ethereumjs that is missing the more helpful error message taylor recently added about the "from" field. |
[R4R]add metrics for contract code bitmap cache
…ntract calls (ethereum#472) * set max gas to double of the charged gas for the 'intrinsic' evm operations * addressed PR comments * addressed pr comment
* 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
* 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
fix: disable mining on follower nodes
* implement post-nyota verkle costs * fix tests and a few bugs * make linter happy * attempt at fixing replay + shadowfork * nyota fixes (ethereum#469) Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com> * fixes from replaying nyota costs (ethereum#472) * fixes from replaying nyota costs Signed-off-by: Guillaume Ballet <3272758+gballet@users.noreply.github.com> * add left-out conversion file * fix balance endianness + set code size in UpdateAccount * add leftover endianness change * remove code cleanup leftover * use 5 as the code size offset this matches the spec, even if we have to add a weird -1 here and there * save buffer creation for PutUint32 --------- Signed-off-by: Guillaume Ballet <3272758+gballet@users.noreply.github.com> * fix most tests * ci: fix buggy target spec tests (temporary) Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com> * fix last failing test --------- Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com> Signed-off-by: Guillaume Ballet <3272758+gballet@users.noreply.github.com> Co-authored-by: Ignacio Hagopian <jsign.uy@gmail.com>
with
develop
commit 11390a9:The text was updated successfully, but these errors were encountered: