Skip to content

qt: allow walletpassphrase in debug console without -server#3553

Merged
laanwj merged 1 commit intobitcoin:masterfrom
laanwj:2014_01_walletpassphrase_qt
Jan 23, 2014
Merged

qt: allow walletpassphrase in debug console without -server#3553
laanwj merged 1 commit intobitcoin:masterfrom
laanwj:2014_01_walletpassphrase_qt

Conversation

@laanwj
Copy link
Copy Markdown
Member

@laanwj laanwj commented Jan 17, 2014

Currently it is only possible to use walletpassphrase to unlock the wallet when bitcoin is started in server mode.

Almost everything that manipulates the wallet in the RPC console (for example, importprivkey) needs the wallet to be unlocked and is thus unusable without -server.

This is pretty unintuitive to me, and I'm sure it's even more confusing to users.

Solve this with a very minimal change: by making the GUI start a dummy RPC thread just to handle deadline timers.

Currently it is only possible to use `walletpassphrase` to unlock the
wallet when bitcoin is started in server mode.

Almost everything that manipulates the wallet in the RPC console
needs the wallet to be unlocked and is thus unusable without -server.

This is pretty unintuitive to me, and I'm sure it's even more confusing
to users.

Solve this with a very minimal change: by making the GUI start a
dummy RPC thread just to handle timeouts.
@BitcoinPullTester
Copy link
Copy Markdown

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/a8db31c83d6a43e07f741f7f61b1bf0df87621c7 for binaries and test log.
This test script verifies pulls every time they are updated. It, however, dies sometimes and fails to test properly. If you are waiting on a test, please check timestamps to verify that the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/
Contact BlueMatt on freenode if something looks broken.

@sipa
Copy link
Copy Markdown
Member

sipa commented Jan 18, 2014

ACK

laanwj added a commit that referenced this pull request Jan 23, 2014
a8db31c qt: allow `walletpassphrase` in debug console without -server (Wladimir J. van der Laan)
@laanwj laanwj merged commit a8db31c into bitcoin:master Jan 23, 2014
@laanwj laanwj deleted the 2014_01_walletpassphrase_qt branch April 9, 2014 14:20
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
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.

3 participants