Skip to content
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

Additional CLI account commands #541

Closed
4 tasks done
obscuren opened this issue Mar 20, 2015 · 13 comments
Closed
4 tasks done

Additional CLI account commands #541

obscuren opened this issue Mar 20, 2015 · 13 comments
Assignees
Milestone

Comments

@obscuren
Copy link
Contributor

https://github.com/ethereum/go-ethereum/wiki/Managing-your-accounts

@obscuren obscuren added this to the Frontier milestone Mar 20, 2015
@obscuren obscuren changed the title Export private keys Additional CLI account commands Mar 20, 2015
@zelig
Copy link
Contributor

zelig commented Mar 20, 2015

how about

ethereum account export <address> <path>
and
ethereum account import <keyfilepath>
i would not encourage people to have their private keys in console histories (esp. if we allow unencrypted format).

@fjl
Copy link
Contributor

fjl commented Mar 20, 2015

Importing from command line is important for system-testing. Maybe both should be supported.

@zelig
Copy link
Contributor

zelig commented Mar 20, 2015

I think you can always get round it ;)

ethereum account import <(echo mynotsosecrettestkey)

@obscuren
Copy link
Contributor Author

I've never seen the importance of supplying a file when you can do cat file (apperantly iPhones haven't got backticks on their keyboard)

@tgerring
Copy link
Contributor

Tap and hold the single quote key--a popover menu should appear with the backpack symbol ;)

@zelig zelig self-assigned this Mar 21, 2015
@zelig
Copy link
Contributor

zelig commented Mar 21, 2015

@obscuren sure you can get round both ways.
Right, how about the best of both worlds: allow filepath only and fall back to stdin if arg missing.
Both passwords and keys . This implements the design principle that the less secure a solution, the more hackery it requires.

Scenarios:

naive user:

  ethereum account import
  please enter your private key:
  please enter a passphrase:
  Remember your passphrase!
  Oh, and dont forget your passphrase!

geek scripting in public cafe or peeps copy/pasting instructions on stackoverflow:

  ethereum --passwd /path/to/passfile account import path/to/.ethkey.prv >  /dev/null 2>&1

tester:

 ethereum --passwd <(echo 'youcanhackme') account import <(echo '34aebf5689fed6....')

or

ethereum account --unencrypted-keys import <(echo '34aebf5689fed6....')

The same for --unlock and export btw

@obscuren
Copy link
Contributor Author

@zelig I think you misunderstand me.

ethereum --import `cat myFile`      # note this doesn't output anything from cat

Is the same as doing (without the need to actually code it up yourself)

ethereum --import-file myFile

Or does that have any other implications?

@tgerring Oh, thanks :-)

@fjl
Copy link
Contributor

fjl commented Mar 21, 2015

                                                                                  argv can be read by anyone, input can't. 

@zelig
Copy link
Contributor

zelig commented Mar 21, 2015

@obscuren No, I understood you. We can do it both ways: direct input is easier to implement, but with file paths you more actively discourage people from showing their key on screen or record in command history. was my point. so?

@zelig
Copy link
Contributor

zelig commented Mar 23, 2015

support for presale wallet import #267 needed too no?

@zelig zelig added ready and removed in progress labels Mar 25, 2015
@subtly
Copy link
Member

subtly commented Mar 26, 2015

Fewer options for users is better. File paths make importing an exported key easier and windows users don't deal much with stdin/stdout.
Ideally export will never be plaintext and decrypting would require a separate command. Import is great for testing whereas the only use-case for export is for users. Is there a reason export would need to be plaintext?

Just my thoughts. Whatever ya'll come up with, having it documented in -help or wiki would be great for system testing :)

@zelig
Copy link
Contributor

zelig commented Mar 26, 2015

@zelig
Copy link
Contributor

zelig commented Apr 1, 2015

all cover in #550 and #580

@zelig zelig closed this as completed Apr 1, 2015
@zelig zelig removed the ready label Apr 1, 2015
nolash pushed a commit to nolash/go-ethereum that referenced this issue May 25, 2018
…efactor

swarm/storage, swarm/api: Content addressed root chunks for MRU
maoueh pushed a commit to streamingfast/go-ethereum that referenced this issue Dec 9, 2022
* add support for bor.logs flag in new-cli

* handle bor.withoutheimdall flag commenting in conversion script
maoueh pushed a commit to streamingfast/go-ethereum that referenced this issue Dec 9, 2022
internal/cli: add support for bor.logs flag in new-cli (ethereum#541)
maoueh pushed a commit to streamingfast/go-ethereum that referenced this issue Jan 6, 2023
* add support for bor.logs flag in new-cli

* handle bor.withoutheimdall flag commenting in conversion script
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants