Skip to content

Commit

Permalink
key: Improve docs
Browse files Browse the repository at this point in the history
Crypto is _hard_. Make an effort to improve the docs with a minimum of
exactly correct information.
  • Loading branch information
tcharding committed Aug 17, 2023
1 parent 14e8218 commit f140750
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions src/key.rs
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,9 @@ use crate::{ecdsa, SECP256K1};
#[cfg(feature = "bitcoin_hashes")]
use crate::{hashes, ThirtyTwoByteHash};

/// Secret 256-bit key used as `x` in an ECDSA signature.
/// Secret key - a 256-bit key used to create ECDSA and schnorr signatures.
///
/// This value should be generated using a [cryptographically secure pseudorandom number generator].
///
/// # Side channel attacks
///
Expand Down Expand Up @@ -48,6 +50,7 @@ use crate::{hashes, ThirtyTwoByteHash};
/// ```
/// [`bincode`]: https://docs.rs/bincode
/// [`cbor`]: https://docs.rs/cbor
/// [cryptographically secure pseudorandom number generator]: https://en.wikipedia.org/wiki/Cryptographically_secure_pseudorandom_number_generator
#[derive(Copy, Clone)]
pub struct SecretKey([u8; constants::SECRET_KEY_SIZE]);
impl_display_secret!(SecretKey);
Expand Down Expand Up @@ -115,7 +118,7 @@ impl str::FromStr for SecretKey {
}
}

/// A Secp256k1 public key, used for verification of signatures.
/// Public key - used to verify ECDSA signatures, for ECDH handshakes, and to do Taproot tweaks.
///
/// # Serde support
///
Expand Down

0 comments on commit f140750

Please sign in to comment.