nostr: require CryptoRng for secret key generation

nostr/CHANGELOG.md

@@ -29,6 +29,10 @@
 
 ## Unreleased
 
+### Breaking changes
+
+- Require `CryptoRng` trait for secret key generation (https://github.com/nostrdevkit/nostr/pull/1437)
+
 ### Fixed
 
 - The `Display` implementations of `NostrConnectMessage` redact the sensitive data (https://github.com/nostrdevkit/nostr/pull/1432)

nostr/examples/embedded/src/main.rs

@@ -14,7 +14,7 @@ use core::panic::PanicInfo;
 use alloc_cortex_m::CortexMHeap;
 use cortex_m_rt::entry;
 use cortex_m_semihosting::{debug, hprintln};
-use rand::TryRng;
+use rand::{TryRng, TryCryptoRng};
 use secp256k1::Secp256k1;
 use nostr::key::{Keys, SecretKey};
 use nostr::nips::nip19::{FromBech32, ToBech32};
@@ -47,6 +47,8 @@ impl TryRng for FakeRng {
     }
 }
 
+impl TryCryptoRng for FakeRng {}
+
 
 #[entry]
 fn main() -> ! {

nostr/src/key/mod.rs

@@ -154,7 +154,7 @@ impl Keys {
     pub fn generate_with_rng<C, R>(secp: &Secp256k1<C>, rng: &mut R) -> Self
     where
         C: Signing,
-        R: Rng,
+        R: Rng + CryptoRng,
     {
         let secret_key: SecretKey = SecretKey::generate_with_rng(rng);
         Self::new_with_ctx(secp, secret_key)

nostr/src/key/secret_key.rs

@@ -8,12 +8,12 @@ use alloc::string::String;
 use core::ops::{Deref, DerefMut};
 use core::str::FromStr;
 
-#[cfg(feature = "rand")]
-use rand::Rng;
 #[cfg(all(feature = "std", feature = "os-rng"))]
 use rand::rand_core::UnwrapErr;
 #[cfg(all(feature = "std", feature = "os-rng"))]
 use rand::rngs::SysRng;
+#[cfg(feature = "rand")]
+use rand::{CryptoRng, Rng};
 use serde::{Deserialize, Deserializer};
 
 use crate::error::{Error, ErrorKind};
@@ -99,12 +99,12 @@ impl SecretKey {
     #[cfg(feature = "rand")]
     pub fn generate_with_rng<R>(rng: &mut R) -> Self
     where
-        R: Rng,
+        R: Rng + CryptoRng,
     {
         let mut data: [u8; Self::LEN] = util::random_32_bytes(rng);
 
         loop {
-            match secp256k1::SecretKey::from_slice(&data) {
+            match secp256k1::SecretKey::from_byte_array(&data) {
                 Ok(secret_key) => return Self { inner: secret_key },
                 Err(_) => data = util::random_32_bytes(rng),
             }