bridge descs: Apply sensitive() to bridge identity in error log msg

This commit is contained in:
Ian Jackson 2022-11-08 13:19:00 +00:00
parent 93a150229a
commit d9d9cdeac9
3 changed files with 4 additions and 1 deletions

1
Cargo.lock generated
View File

@ -3696,6 +3696,7 @@ dependencies = [
"rand 0.8.5",
"retry-error",
"rusqlite",
"safelog",
"scopeguard",
"serde",
"signature",

View File

@ -51,6 +51,7 @@ postage = { version = "0.5.0", default-features = false, features = ["futures-tr
rand = "0.8"
retry-error = { path = "../retry-error", version = "0.2.0" }
rusqlite = { version = "0.28.0", features = ["time"] }
safelog = { path = "../safelog", version = "0.1.2" }
scopeguard = "1"
serde = { version = "1.0.103", features = ["derive"] }
signature = "1"

View File

@ -20,6 +20,7 @@ use futures::task::{SpawnError, SpawnExt as _};
use futures::FutureExt;
use tracing::{debug, error, info, trace};
use safelog::sensitive;
use tor_basic_utils::retry::RetryDelay;
use tor_basic_utils::BinaryHeapExt as _;
use tor_checkable::{SelfSigned, Timebound};
@ -954,7 +955,7 @@ impl<R: Runtime, M: Mockable<R>> Manager<R, M> {
.unwrap_or_else(|err| {
error!(
r#"bridge descriptor cache lookup failed, for "{}": {}"#,
bridge,
sensitive(bridge),
tor_error::Report(&err)
);
None