Skip to content

Commit 758747a

Browse files
committed
Drop Debug on InMemorySigner (and EntropySource)
`InMemorySigner` has various private keys in it which makes `Debug` either useless or dangerous (because most keys won't log anything, but if they did we'd risk logging private key material).
1 parent d35239c commit 758747a

File tree

1 file changed

+0
-2
lines changed

1 file changed

+0
-2
lines changed

lightning/src/sign/mod.rs

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1027,7 +1027,6 @@ pub trait ChangeDestinationSource {
10271027
///
10281028
/// This implementation performs no policy checks and is insufficient by itself as
10291029
/// a secure external signer.
1030-
#[derive(Debug)]
10311030
pub struct InMemorySigner {
10321031
/// Holder secret key in the 2-of-2 multisig script of a channel. This key also backs the
10331032
/// holder's anchor output in a commitment transaction, if one is present.
@@ -2475,7 +2474,6 @@ impl PhantomKeysManager {
24752474
}
24762475

24772476
/// An implementation of [`EntropySource`] using ChaCha20.
2478-
#[derive(Debug)]
24792477
pub struct RandomBytes {
24802478
/// Seed from which all randomness produced is derived from.
24812479
seed: [u8; 32],

0 commit comments

Comments
 (0)