Skip to content

Commit 7305503

Browse files
committed
fix: stop deriving Debug for Endpoint
Endpoint contains BMC credentials, including password, so deriving Debug makes it easier to accidentally expose secrets in logs or error output. Remove the Debug derive from Endpoint to avoid accidental credential leakage. Signed-off-by: Dmitry Porokh <dporokh@nvidia.com>
1 parent b470bef commit 7305503

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

src/network.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,7 @@ impl RedfishClientPoolBuilder {
9797
}
9898

9999
/// The endpoint that the redfish client connects to
100-
#[derive(Debug, Clone, PartialEq, Eq)]
100+
#[derive(Clone, PartialEq, Eq)] // WARN: Do not derive Debug: Endpoint may contain credentials and must not be logged accidentally.
101101
pub struct Endpoint {
102102
/// Hostname or IP address of BMC
103103
pub host: String,

0 commit comments

Comments
 (0)