Skip to content

Commit

Permalink
Replace trace() message with error() in main.rs
Browse files Browse the repository at this point in the history
For error messages like "wallet file not found" or
"bitcoin node unreachable" it is much more appropriate to use error()
rather than trace()
  • Loading branch information
chris-belcher committed Nov 25, 2021
1 parent 5f6283e commit cc401f0
Showing 1 changed file with 10 additions and 10 deletions.
20 changes: 10 additions & 10 deletions src/main.rs
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ fn generate_wallet(wallet_file_name: &PathBuf) -> std::io::Result<()> {
let rpc = match get_bitcoin_rpc() {
Ok(rpc) => rpc,
Err(error) => {
log::trace!(target: "main", "error connecting to bitcoin node: {:?}", error);
log::error!(target: "main", "error connecting to bitcoin node: {:?}", error);
return Ok(());
}
};
Expand Down Expand Up @@ -125,14 +125,14 @@ fn display_wallet_balance(wallet_file_name: &PathBuf, long_form: Option<bool>) {
let mut wallet = match Wallet::load_wallet_from_file(wallet_file_name) {
Ok(w) => w,
Err(error) => {
log::trace!(target: "main", "error loading wallet file: {:?}", error);
log::error!(target: "main", "error loading wallet file: {:?}", error);
return;
}
};
let rpc = match get_bitcoin_rpc() {
Ok(rpc) => rpc,
Err(error) => {
log::trace!(target: "main", "error connecting to bitcoin node: {:?}", error);
log::error!(target: "main", "error connecting to bitcoin node: {:?}", error);
return;
}
};
Expand Down Expand Up @@ -260,7 +260,7 @@ fn display_wallet_keys(wallet_file_name: &PathBuf) {
let wallet = match Wallet::load_wallet_from_file(wallet_file_name) {
Ok(w) => w,
Err(error) => {
log::trace!(target: "main", "error loading wallet file: {:?}", error);
log::error!(target: "main", "error loading wallet file: {:?}", error);
return;
}
};
Expand All @@ -271,14 +271,14 @@ fn print_receive_invoice(wallet_file_name: &PathBuf) {
let mut wallet = match Wallet::load_wallet_from_file(wallet_file_name) {
Ok(w) => w,
Err(error) => {
log::trace!(target: "main", "error loading wallet file: {:?}", error);
log::error!(target: "main", "error loading wallet file: {:?}", error);
return;
}
};
let rpc = match get_bitcoin_rpc() {
Ok(rpc) => rpc,
Err(error) => {
log::trace!(target: "main", "error connecting to bitcoin node: {:?}", error);
log::error!(target: "main", "error connecting to bitcoin node: {:?}", error);
return;
}
};
Expand All @@ -298,14 +298,14 @@ fn run_maker(wallet_file_name: &PathBuf, port: u16) {
let rpc = match get_bitcoin_rpc() {
Ok(rpc) => rpc,
Err(error) => {
log::trace!(target: "main", "error connecting to bitcoin node: {:?}", error);
log::error!(target: "main", "error connecting to bitcoin node: {:?}", error);
return;
}
};
let mut wallet = match Wallet::load_wallet_from_file(wallet_file_name) {
Ok(w) => w,
Err(error) => {
log::trace!(target: "main", "error loading wallet file: {:?}", error);
log::error!(target: "main", "error loading wallet file: {:?}", error);
return;
}
};
Expand All @@ -320,14 +320,14 @@ fn run_taker(wallet_file_name: &PathBuf) {
let rpc = match get_bitcoin_rpc() {
Ok(rpc) => rpc,
Err(error) => {
log::trace!(target: "main", "error connecting to bitcoin node: {:?}", error);
log::error!(target: "main", "error connecting to bitcoin node: {:?}", error);
return;
}
};
let mut wallet = match Wallet::load_wallet_from_file(wallet_file_name) {
Ok(w) => w,
Err(error) => {
log::trace!(target: "main", "error loading wallet file: {:?}", error);
log::error!(target: "main", "error loading wallet file: {:?}", error);
return;
}
};
Expand Down

0 comments on commit cc401f0

Please sign in to comment.