Make `reveal` print the secret key, not the public one.

That's what the method documentation says. It explicitly warns that it
reveals the secret key, unlike the Debug implementation.
master
Andreas Fackler 5 years ago committed by Andreas Fackler
parent 2d332a82bd
commit 16a3bb650c
  1. 6
      src/lib.rs

@ -402,8 +402,7 @@ impl SecretKey {
/// the `Debug` implementation in that it *does* leak the secret prime /// the `Debug` implementation in that it *does* leak the secret prime
/// field element. /// field element.
pub fn reveal(&self) -> String { pub fn reveal(&self) -> String {
let uncomp = self.public_key().0.into_affine().into_uncompressed(); format!("SecretKey({:?})", self.0)
format!("SecretKey({:0.10})", HexFmt(uncomp))
} }
} }
@ -474,8 +473,7 @@ impl SecretKeyShare {
/// the `Debug` implementation in that it *does* leak the secret prime /// the `Debug` implementation in that it *does* leak the secret prime
/// field element. /// field element.
pub fn reveal(&self) -> String { pub fn reveal(&self) -> String {
let uncomp = self.0.public_key().0.into_affine().into_uncompressed(); format!("SecretKeyShare({:?})", (self.0).0)
format!("SecretKeyShare({:0.10})", HexFmt(uncomp))
} }
} }

Loading…
Cancel
Save