Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Remove wallet unlock unlock-timeout option #4139

Merged
merged 1 commit into from
Jun 15, 2018

Conversation

heifner
Copy link
Contributor

@heifner heifner commented Jun 15, 2018

cleos wallet unlock has a --unlock-timeout option but this option only does something if the wallet is not already running. If you do anything else first, like say cleos wallet open this is silently ignored.

Remove this option until it can be supported via rpc interface on keosd. This could possibly confuse users into thinking their wallet will be locked quickly when it will not be.

@heifner heifner requested a review from arhag June 15, 2018 14:14
@arhag arhag added this to the Version 1.0.4 milestone Jun 15, 2018
@arhag arhag merged commit 9f25a5b into master Jun 15, 2018
@arhag arhag deleted the gh#4136-cleos-unlock-timeout branch June 15, 2018 15:07
@coolmagic
Copy link

Is there any way I can keep wallet open all the time? Running local node

@jgiszczak
Copy link
Contributor

@coolmagic You may specify a very long unlock-timeout= configuration option. The type is std::chrono::seconds which according to the specification should be able to hold a duration up to 292 years.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants