]> git.bitcoin.ninja Git - rust-lightning/commitdiff
Drop `Debug` on `InMemorySigner` (and `EntropySource`)
authorMatt Corallo <git@bluematt.me>
Wed, 11 Sep 2024 20:00:26 +0000 (20:00 +0000)
committerMatt Corallo <git@bluematt.me>
Wed, 11 Sep 2024 20:01:43 +0000 (20:01 +0000)
`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).

lightning/src/sign/mod.rs

index 0d3ec9efa417863fcd9bf9f37a901f5a63df68b5..890c1bc0fd27d48b7816e510da4b96e040b4f397 100644 (file)
@@ -1027,7 +1027,6 @@ pub trait ChangeDestinationSource {
 ///
 /// This implementation performs no policy checks and is insufficient by itself as
 /// a secure external signer.
-#[derive(Debug)]
 pub struct InMemorySigner {
        /// Holder secret key in the 2-of-2 multisig script of a channel. This key also backs the
        /// holder's anchor output in a commitment transaction, if one is present.
@@ -2475,7 +2474,6 @@ impl PhantomKeysManager {
 }
 
 /// An implementation of [`EntropySource`] using ChaCha20.
-#[derive(Debug)]
 pub struct RandomBytes {
        /// Seed from which all randomness produced is derived from.
        seed: [u8; 32],