From 3ba1ed2bcb3301a59375f4acb7916d1a7d7d76ca Mon Sep 17 00:00:00 2001 From: rishflab Date: Sat, 11 Sep 2021 22:17:32 +1000 Subject: [PATCH] Change message log level to debug to allow enabling through flag We do not have a way to enable tracing through a command line argument so it did not make sense to have these messages set to trace. Ideally a trace flag should be added but it is not that straightforward with structopt. We could add a --log-level arg that allows you select a log level but this is verbose. --- CHANGELOG.md | 1 + swap/src/network/tor_transport.rs | 4 ++-- swap/src/protocol/bob/swap.rs | 2 +- 3 files changed, 4 insertions(+), 3 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 5f205f00..28e512a5 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -12,6 +12,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 - The `cancel`, `refund` and `punish` subcommands in ASB and CLI are run with the `--force` by default and the `--force` option has been removed. The force flag was used to ignore blockheight and protocol state checks. Users can still restart a swap with these checks using the `resume` subcommand. +- Changed log level of the "Advancing state", "Establishing Connection through Tor proxy" and "Connection through Tor established" log message from tracing to debug in the CLI. ## [0.8.3] - 2021-09-03 diff --git a/swap/src/network/tor_transport.rs b/swap/src/network/tor_transport.rs index 04484cd3..2bee690a 100644 --- a/swap/src/network/tor_transport.rs +++ b/swap/src/network/tor_transport.rs @@ -42,14 +42,14 @@ impl Transport for TorDialOnlyTransport { } let dial_future = async move { - tracing::trace!(address = %addr, "Establishing connection through Tor proxy"); + tracing::debug!(address = %addr, "Establishing connection through Tor proxy"); let stream = Socks5Stream::connect((Ipv4Addr::LOCALHOST, self.socks_port), address.to_string()) .await .map_err(|e| io::Error::new(io::ErrorKind::ConnectionRefused, e))?; - tracing::trace!("Connection through Tor established"); + tracing::debug!("Connection through Tor established"); Ok(TcpStream(stream.into_inner())) }; diff --git a/swap/src/protocol/bob/swap.rs b/swap/src/protocol/bob/swap.rs index 1369376c..e1e9f849 100644 --- a/swap/src/protocol/bob/swap.rs +++ b/swap/src/protocol/bob/swap.rs @@ -58,7 +58,7 @@ async fn next_state( monero_wallet: &monero::Wallet, monero_receive_address: monero::Address, ) -> Result { - tracing::trace!(%state, "Advancing state"); + tracing::debug!(%state, "Advancing state"); Ok(match state { BobState::Started {