Skip to content

Commit

Permalink
rebase 5.3.X [PATCH 111/142] Do not prompt for password to start owne…
Browse files Browse the repository at this point in the history
…r API (mimblewimble#656)

* Do not prompt for password to start owner API
  It makes no sense to ask for the password if one only wants to initialize the APIs. Asking for the password makes things unnecessarily difficult and insecure in case I, as a system administrator, want to have the APIs running as a system service.
* Update wallet_args.rs
* Update wallet_args.rs
   opening wallet if password argument is present
  • Loading branch information
bayk committed Aug 9, 2024
1 parent f272771 commit e71b38f
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions src/cmd/wallet_args.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1497,10 +1497,11 @@ where
("recover", _) => open_wallet = false,
("cli", _) => open_wallet = false,
("owner_api", _) => {
// If wallet exists, open it. Otherwise, that's fine too.
// If wallet exists and password is present then open it. Otherwise, that's fine too.
let mut wallet_lock = wallet.lock();
let lc = wallet_lock.lc_provider().unwrap();
open_wallet = lc.wallet_exists(None, wallet_config.wallet_data_dir.as_deref())?;
open_wallet = wallet_args.is_present("pass")
&& lc.wallet_exists(None, wallet_config.wallet_data_dir.as_deref())?;
}
_ => {}
}
Expand Down

0 comments on commit e71b38f

Please sign in to comment.