diff --git a/bitcoin/src/address/mod.rs b/bitcoin/src/address/mod.rs index ae48a9b3f..d05d0cdf3 100644 --- a/bitcoin/src/address/mod.rs +++ b/bitcoin/src/address/mod.rs @@ -194,7 +194,7 @@ impl fmt::Display for AddressInner { pub enum KnownHrp { /// The main Bitcoin network. Mainnet, - /// The test networks, testnet and signet. + /// The test networks, testnet (testnet3), testnet4, and signet. Testnets, /// The regtest network. Regtest, @@ -207,7 +207,7 @@ impl KnownHrp { match network { Bitcoin => Self::Mainnet, - Testnet | Signet => Self::Testnets, + Testnet(_) | Signet => Self::Testnets, Regtest => Self::Regtest, } } @@ -708,11 +708,11 @@ impl Address { /// network a simple comparison is not enough anymore. Instead this function can be used. /// /// ```rust - /// use bitcoin::{Address, Network}; + /// use bitcoin::{Address, Network, TestnetVersion}; /// use bitcoin::address::NetworkUnchecked; /// /// let address: Address = "2N83imGV3gPwBzKJQvWJ7cRUY2SpUyU6A5e".parse().unwrap(); - /// assert!(address.is_valid_for_network(Network::Testnet)); + /// assert!(address.is_valid_for_network(Network::Testnet(TestnetVersion::V3))); /// assert!(address.is_valid_for_network(Network::Regtest)); /// assert!(address.is_valid_for_network(Network::Signet)); /// @@ -720,7 +720,7 @@ impl Address { /// /// let address: Address = "32iVBEu4dxkUQk9dJbZUiBiQdmypcEyJRf".parse().unwrap(); /// assert!(address.is_valid_for_network(Network::Bitcoin)); - /// assert_eq!(address.is_valid_for_network(Network::Testnet), false); + /// assert_eq!(address.is_valid_for_network(Network::Testnet(TestnetVersion::V4)), false); /// ``` pub fn is_valid_for_network(&self, n: Network) -> bool { use AddressInner::*; @@ -888,8 +888,8 @@ mod tests { use hex_lit::hex; use super::*; - use crate::network::params; use crate::network::Network::{Bitcoin, Testnet}; + use crate::network::{params, TestnetVersion}; use crate::script::ScriptBufExt as _; fn roundtrips(addr: &Address, network: Network) { @@ -942,7 +942,7 @@ mod tests { let addr = Address::p2pkh(key, NetworkKind::Test); assert_eq!(&addr.to_string(), "mqkhEMH6NCeYjFybv7pvFC22MFeaNT9AQC"); assert_eq!(addr.address_type(), Some(AddressType::P2pkh)); - roundtrips(&addr, Testnet); + roundtrips(&addr, Testnet(TestnetVersion::V3)); } #[test] @@ -965,7 +965,7 @@ mod tests { let addr = Address::p2sh(&script, NetworkKind::Test).unwrap(); assert_eq!(&addr.to_string(), "2N3zXjbwdTcPsJiy8sUK9FhWJhqQCxA8Jjr"); assert_eq!(addr.address_type(), Some(AddressType::P2sh)); - roundtrips(&addr, Testnet); + roundtrips(&addr, Testnet(TestnetVersion::V3)); } #[test] @@ -1282,7 +1282,7 @@ mod tests { let address = address_string .parse::>() .expect("address") - .require_network(Network::Testnet) + .require_network(Network::Testnet(TestnetVersion::V3)) .expect("testnet"); let pubkey_string = "04e96e22004e3db93530de27ccddfdf1463975d2138ac018fc3e7ba1a2e5e0aad8e424d0b55e2436eb1d0dcd5cb2b8bcc6d53412c22f358de57803a6a655fbbd04"; diff --git a/bitcoin/src/blockdata/block.rs b/bitcoin/src/blockdata/block.rs index c5ddf11fa..9b9e8f2d8 100644 --- a/bitcoin/src/blockdata/block.rs +++ b/bitcoin/src/blockdata/block.rs @@ -477,7 +477,7 @@ mod tests { use super::*; use crate::consensus::encode::{deserialize, serialize}; - use crate::Network; + use crate::{Network, TestnetVersion}; #[test] fn test_coinbase_and_bip34() { @@ -546,7 +546,7 @@ mod tests { // Check testnet block 000000000000045e0b1660b6445b5e5c5ab63c9a4f956be7e1e69be04fa4497b #[test] fn segwit_block_test() { - let params = Params::new(Network::Testnet); + let params = Params::new(Network::Testnet(TestnetVersion::V3)); let segwit_block = include_bytes!("../../tests/data/testnet_block_000000000000045e0b1660b6445b5e5c5ab63c9a4f956be7e1e69be04fa4497b.raw").to_vec(); let decode: Result = deserialize(&segwit_block); diff --git a/bitcoin/src/blockdata/constants.rs b/bitcoin/src/blockdata/constants.rs index 8e828304b..a891ba038 100644 --- a/bitcoin/src/blockdata/constants.rs +++ b/bitcoin/src/blockdata/constants.rs @@ -17,7 +17,7 @@ use crate::opcodes::all::*; use crate::pow::CompactTarget; use crate::transaction::{self, OutPoint, Transaction, TxIn, TxOut}; use crate::witness::Witness; -use crate::{script, Amount, BlockHash, Sequence}; +use crate::{script, Amount, BlockHash, Sequence, TestnetVersion}; /// How many seconds between blocks we expect on average. pub const TARGET_BLOCK_SPACING: u32 = 600; @@ -72,8 +72,11 @@ const GENESIS_OUTPUT_PK: [u8; 65] = [ 0x8a, 0x4c, 0x70, 0x2b, 0x6b, 0xf1, 0x1d, 0x5f ]; +#[rustfmt::skip] +const TESTNET4_GENESIS_OUTPUT_PK: [u8; 33] = [0x00; 33]; + /// Constructs and returns the coinbase (and only) transaction of the Bitcoin genesis block. -fn bitcoin_genesis_tx() -> Transaction { +fn bitcoin_genesis_tx(params: &Params) -> Transaction { // Base let mut ret = Transaction { version: transaction::Version::ONE, @@ -82,12 +85,28 @@ fn bitcoin_genesis_tx() -> Transaction { output: vec![], }; - // Inputs - let in_script = script::Builder::new() - .push_int_unchecked(486604799) - .push_int_non_minimal(4) - .push_slice(b"The Times 03/Jan/2009 Chancellor on brink of second bailout for banks") - .into_script(); + let (in_script, out_script) = { + match params.network { + Network::Testnet(TestnetVersion::V4) => ( + script::Builder::new() + .push_int_unchecked(486604799) + .push_int_non_minimal(4) + .push_slice(b"03/May/2024 000000000000000000001ebd58c244970b3aa9d783bb001011fbe8ea8e98e00e") + .into_script(), + script::Builder::new().push_slice(TESTNET4_GENESIS_OUTPUT_PK).push_opcode(OP_CHECKSIG).into_script(), + + ), + _ => ( + script::Builder::new() + .push_int_unchecked(486604799) + .push_int_non_minimal(4) + .push_slice(b"The Times 03/Jan/2009 Chancellor on brink of second bailout for banks") + .into_script(), + script::Builder::new().push_slice(GENESIS_OUTPUT_PK).push_opcode(OP_CHECKSIG).into_script(), + ), + } + }; + ret.input.push(TxIn { previous_output: OutPoint::COINBASE_PREVOUT, script_sig: in_script, @@ -95,9 +114,6 @@ fn bitcoin_genesis_tx() -> Transaction { witness: Witness::default(), }); - // Outputs - let out_script = - script::Builder::new().push_slice(GENESIS_OUTPUT_PK).push_opcode(OP_CHECKSIG).into_script(); ret.output.push(TxOut { value: Amount::from_sat(50 * 100_000_000), script_pubkey: out_script }); // end @@ -106,10 +122,12 @@ fn bitcoin_genesis_tx() -> Transaction { /// Constructs and returns the genesis block. pub fn genesis_block(params: impl AsRef) -> Block { - let txdata = vec![bitcoin_genesis_tx()]; + let params = params.as_ref(); + let txdata = vec![bitcoin_genesis_tx(params)]; let hash: sha256d::Hash = txdata[0].compute_txid().into(); - let merkle_root = hash.into(); - match params.as_ref().network { + let merkle_root: crate::TxMerkleNode = hash.into(); + + match params.network { Network::Bitcoin => Block { header: block::Header { version: block::Version::ONE, @@ -121,7 +139,7 @@ pub fn genesis_block(params: impl AsRef) -> Block { }, txdata, }, - Network::Testnet => Block { + Network::Testnet(TestnetVersion::V3) => Block { header: block::Header { version: block::Version::ONE, prev_blockhash: BlockHash::GENESIS_PREVIOUS_BLOCK_HASH, @@ -132,6 +150,17 @@ pub fn genesis_block(params: impl AsRef) -> Block { }, txdata, }, + Network::Testnet(TestnetVersion::V4) => Block { + header: block::Header { + version: block::Version::ONE, + prev_blockhash: BlockHash::GENESIS_PREVIOUS_BLOCK_HASH, + merkle_root, + time: 1714777860, + bits: CompactTarget::from_consensus(0x1d00ffff), + nonce: 393743547, + }, + txdata, + }, Network::Signet => Block { header: block::Header { version: block::Version::ONE, @@ -170,11 +199,22 @@ impl ChainHash { 111, 226, 140, 10, 182, 241, 179, 114, 193, 166, 162, 70, 174, 99, 247, 79, 147, 30, 131, 101, 225, 90, 8, 156, 104, 214, 25, 0, 0, 0, 0, 0, ]); - /// `ChainHash` for testnet bitcoin. + /// `ChainHash` for testnet3 bitcoin. + #[deprecated(since = "0.33.0", note = "Use TESTNET3 instead")] pub const TESTNET: Self = Self([ 67, 73, 127, 215, 248, 38, 149, 113, 8, 244, 163, 15, 217, 206, 195, 174, 186, 121, 151, 32, 132, 233, 14, 173, 1, 234, 51, 9, 0, 0, 0, 0, ]); + /// `ChainHash` for testnet3 bitcoin. + pub const TESTNET3: Self = Self([ + 67, 73, 127, 215, 248, 38, 149, 113, 8, 244, 163, 15, 217, 206, 195, 174, 186, 121, 151, + 32, 132, 233, 14, 173, 1, 234, 51, 9, 0, 0, 0, 0, + ]); + /// `ChainHash` for testnet4 bitcoin. + pub const TESTNET4: Self = Self([ + 67, 240, 139, 218, 176, 80, 227, 91, 86, 124, 134, 75, 145, 244, 127, 80, 174, 114, 90, + 226, 222, 83, 188, 251, 186, 242, 132, 218, 0, 0, 0, 0, + ]); /// `ChainHash` for signet bitcoin. pub const SIGNET: Self = Self([ 246, 30, 238, 59, 99, 163, 128, 164, 119, 160, 99, 175, 50, 178, 187, 201, 124, 159, 249, @@ -191,9 +231,13 @@ impl ChainHash { /// See [BOLT 0](https://github.com/lightning/bolts/blob/ffeece3dab1c52efdb9b53ae476539320fa44938/00-introduction.md#chain_hash) /// for specification. pub fn using_genesis_block(params: impl AsRef) -> Self { - let network = params.as_ref().network; - let hashes = [Self::BITCOIN, Self::TESTNET, Self::SIGNET, Self::REGTEST]; - hashes[network as usize] + match params.as_ref().network { + Network::Bitcoin => Self::BITCOIN, + Network::Testnet(TestnetVersion::V3) => Self::TESTNET3, + Network::Testnet(TestnetVersion::V4) => Self::TESTNET4, + Network::Signet => Self::SIGNET, + Network::Regtest => Self::REGTEST, + } } /// Returns the hash of the `network` genesis block for use as a chain hash. @@ -201,8 +245,13 @@ impl ChainHash { /// See [BOLT 0](https://github.com/lightning/bolts/blob/ffeece3dab1c52efdb9b53ae476539320fa44938/00-introduction.md#chain_hash) /// for specification. pub const fn using_genesis_block_const(network: Network) -> Self { - let hashes = [Self::BITCOIN, Self::TESTNET, Self::SIGNET, Self::REGTEST]; - hashes[network as usize] + match network { + Network::Bitcoin => Self::BITCOIN, + Network::Testnet(TestnetVersion::V3) => Self::TESTNET3, + Network::Testnet(TestnetVersion::V4) => Self::TESTNET4, + Network::Signet => Self::SIGNET, + Network::Regtest => Self::REGTEST, + } } /// Converts genesis block hash into `ChainHash`. @@ -222,7 +271,7 @@ mod test { #[test] fn bitcoin_genesis_first_transaction() { - let gen = bitcoin_genesis_tx(); + let gen = bitcoin_genesis_tx(&Params::MAINNET); assert_eq!(gen.version, transaction::Version::ONE); assert_eq!(gen.input.len(), 1); @@ -278,7 +327,7 @@ mod test { #[test] fn testnet_genesis_full_block() { - let gen = genesis_block(¶ms::TESTNET); + let gen = genesis_block(¶ms::TESTNET3); assert_eq!(gen.header.version, block::Version::ONE); assert_eq!(gen.header.prev_blockhash, BlockHash::GENESIS_PREVIOUS_BLOCK_HASH); assert_eq!( @@ -332,7 +381,8 @@ mod test { #[allow(unreachable_patterns)] // This is specifically trying to catch later added variants. match network { Network::Bitcoin => {}, - Network::Testnet => {}, + Network::Testnet(TestnetVersion::V3) => {}, + Network::Testnet(TestnetVersion::V4) => {}, Network::Signet => {}, Network::Regtest => {}, _ => panic!("update ChainHash::using_genesis_block and chain_hash_genesis_block with new variants"), @@ -352,7 +402,8 @@ mod test { chain_hash_genesis_block! { mainnet_chain_hash_genesis_block, Network::Bitcoin; - testnet_chain_hash_genesis_block, Network::Testnet; + testnet_chain_hash_genesis_block, Network::Testnet(TestnetVersion::V3); + testnet4_chain_hash_genesis_block, Network::Testnet(TestnetVersion::V4); signet_chain_hash_genesis_block, Network::Signet; regtest_chain_hash_genesis_block, Network::Regtest; } diff --git a/bitcoin/src/blockdata/script/push_bytes.rs b/bitcoin/src/blockdata/script/push_bytes.rs index 89cbdc268..654342b14 100644 --- a/bitcoin/src/blockdata/script/push_bytes.rs +++ b/bitcoin/src/blockdata/script/push_bytes.rs @@ -202,12 +202,12 @@ mod primitive { } } - // Sizes up to 73 to support all pubkey and signature sizes + // Sizes up to 76 to support all pubkey and signature sizes from_array! { 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20, 21, 22, 23, 24, 25, 26, 27, 28, 29, 30, 31, 32, 33, 34, 35, 36, 37, 38, 39, 40, 41, 42, 43, 44, 45, 46, 47, 48, 49, 50, 51, 52, 53, 54, 55, 56, 57, 58, 59, 60, 61, 62, 63, 64, 65, 66, 67, 68, 69, 70, - 71, 72, 73, + 71, 72, 73, 74, 75, 76 } /// Owned, growable counterpart to `PushBytes`. diff --git a/bitcoin/src/lib.rs b/bitcoin/src/lib.rs index 46f352a95..a030bf3ea 100644 --- a/bitcoin/src/lib.rs +++ b/bitcoin/src/lib.rs @@ -135,7 +135,7 @@ pub use crate::{ crypto::key::{self, PrivateKey, PubkeyHash, PublicKey, CompressedPublicKey, WPubkeyHash, XOnlyPublicKey}, crypto::sighash::{self, LegacySighash, SegwitV0Sighash, TapSighash, TapSighashTag}, merkle_tree::{MerkleBlock, TxMerkleNode, WitnessMerkleNode}, - network::{Network, NetworkKind}, + network::{Network, NetworkKind, TestnetVersion}, network::params::{self, Params}, pow::{CompactTarget, Target, Work}, psbt::Psbt, diff --git a/bitcoin/src/network/mod.rs b/bitcoin/src/network/mod.rs index 26c517ab3..c34ee1c08 100644 --- a/bitcoin/src/network/mod.rs +++ b/bitcoin/src/network/mod.rs @@ -25,7 +25,7 @@ use core::str::FromStr; use internals::write_err; #[cfg(feature = "serde")] -use serde::{Deserialize, Serialize}; +use serde::{de::Visitor, Deserialize, Deserializer, Serialize, Serializer}; use crate::constants::ChainHash; use crate::p2p::Magic; @@ -57,27 +57,72 @@ impl From for NetworkKind { match n { Bitcoin => NetworkKind::Main, - Testnet | Signet | Regtest => NetworkKind::Test, + Testnet(_) | Signet | Regtest => NetworkKind::Test, } } } +/// The testnet version to act on. +#[derive(Copy, PartialEq, Eq, PartialOrd, Ord, Clone, Hash, Debug)] +#[non_exhaustive] +pub enum TestnetVersion { + /// Testnet version 3. + V3, + /// Testnet version 4. + V4, +} + /// The cryptocurrency network to act on. #[derive(Copy, PartialEq, Eq, PartialOrd, Ord, Clone, Hash, Debug)] -#[cfg_attr(feature = "serde", derive(Serialize, Deserialize))] -#[cfg_attr(feature = "serde", serde(rename_all = "lowercase"))] #[non_exhaustive] pub enum Network { /// Mainnet Bitcoin. Bitcoin, /// Bitcoin's testnet network. - Testnet, + Testnet(TestnetVersion), /// Bitcoin's signet network. Signet, /// Bitcoin's regtest network. Regtest, } +#[cfg(feature = "serde")] +impl Serialize for Network { + fn serialize(&self, serializer: S) -> Result + where + S: Serializer, + { + serializer.serialize_str(self.as_display_str()) + } +} + +#[cfg(feature = "serde")] +impl<'de> Deserialize<'de> for Network { + fn deserialize(deserializer: D) -> Result + where + D: Deserializer<'de>, + { + struct NetworkVisitor; + + impl<'de> Visitor<'de> for NetworkVisitor { + type Value = Network; + + fn expecting(&self, formatter: &mut fmt::Formatter) -> fmt::Result { + formatter.write_str("a valid network identifier") + } + + fn visit_str(self, value: &str) -> Result + where + E: serde::de::Error, + { + Network::from_str(value).map_err(E::custom) + } + } + + deserializer.deserialize_str(NetworkVisitor) + } +} + impl Network { /// Creates a `Network` from the magic bytes. /// @@ -118,7 +163,9 @@ impl Network { pub fn to_core_arg(self) -> &'static str { match self { Network::Bitcoin => "main", - Network::Testnet => "test", + // For user-side compatibility, testnet3 is retained as test + Network::Testnet(TestnetVersion::V3) => "test", + Network::Testnet(TestnetVersion::V4) => "testnet4", Network::Signet => "signet", Network::Regtest => "regtest", } @@ -138,7 +185,8 @@ impl Network { let network = match core_arg { "main" => Bitcoin, - "test" => Testnet, + "test" => Testnet(TestnetVersion::V3), + "testnet4" => Testnet(TestnetVersion::V4), "signet" => Signet, "regtest" => Regtest, _ => return Err(ParseNetworkError(core_arg.to_owned())), @@ -175,13 +223,25 @@ impl Network { /// Returns the associated network parameters. pub const fn params(self) -> &'static Params { - const PARAMS: [Params; 4] = [ - Params::new(Network::Bitcoin), - Params::new(Network::Testnet), - Params::new(Network::Signet), - Params::new(Network::Regtest), - ]; - &PARAMS[self as usize] + match self { + Network::Bitcoin => &Params::BITCOIN, + Network::Testnet(TestnetVersion::V3) => &Params::TESTNET3, + Network::Testnet(TestnetVersion::V4) => &Params::TESTNET4, + Network::Signet => &Params::SIGNET, + Network::Regtest => &Params::REGTEST, + } + } + + /// Returns a string representation of the `Network` enum variant. + /// This is useful for displaying the network type as a string. + const fn as_display_str(self) -> &'static str { + match self { + Network::Bitcoin => "bitcoin", + Network::Testnet(TestnetVersion::V3) => "testnet", + Network::Testnet(TestnetVersion::V4) => "testnet4", + Network::Signet => "signet", + Network::Regtest => "regtest", + } } } @@ -212,7 +272,7 @@ pub mod as_core_arg { Network::from_core_arg(s).map_err(|_| { E::invalid_value( serde::de::Unexpected::Str(s), - &"bitcoin network encoded as a string (either main, test, signet or regtest)", + &"bitcoin network encoded as a string (either main, test, testnet4, signet or regtest)", ) }) } @@ -220,7 +280,7 @@ pub mod as_core_arg { fn expecting(&self, formatter: &mut core::fmt::Formatter) -> core::fmt::Result { write!( formatter, - "bitcoin network encoded as a string (either main, test, signet or regtest)" + "bitcoin network encoded as a string (either main, test, testnet4, signet or regtest)" ) } } @@ -250,30 +310,21 @@ impl FromStr for Network { #[inline] fn from_str(s: &str) -> Result { - use Network::*; - - let network = match s { - "bitcoin" => Bitcoin, - "testnet" => Testnet, - "signet" => Signet, - "regtest" => Regtest, - _ => return Err(ParseNetworkError(s.to_owned())), - }; - Ok(network) + match s { + "bitcoin" => Ok(Network::Bitcoin), + // For user-side compatibility, testnet3 is retained as testnet + "testnet" => Ok(Network::Testnet(TestnetVersion::V3)), + "testnet4" => Ok(Network::Testnet(TestnetVersion::V4)), + "signet" => Ok(Network::Signet), + "regtest" => Ok(Network::Regtest), + _ => Err(ParseNetworkError(s.to_owned())), + } } } impl fmt::Display for Network { fn fmt(&self, f: &mut core::fmt::Formatter) -> core::fmt::Result { - use Network::*; - - let s = match *self { - Bitcoin => "bitcoin", - Testnet => "testnet", - Signet => "signet", - Regtest => "regtest", - }; - write!(f, "{}", s) + write!(f, "{}", self.as_display_str()) } } @@ -300,7 +351,8 @@ impl TryFrom for Network { match chain_hash { // Note: any new network entries must be matched against here. ChainHash::BITCOIN => Ok(Network::Bitcoin), - ChainHash::TESTNET => Ok(Network::Testnet), + ChainHash::TESTNET3 => Ok(Network::Testnet(TestnetVersion::V3)), + ChainHash::TESTNET4 => Ok(Network::Testnet(TestnetVersion::V4)), ChainHash::SIGNET => Ok(Network::Signet), ChainHash::REGTEST => Ok(Network::Regtest), _ => Err(UnknownChainHashError(chain_hash)), @@ -310,19 +362,33 @@ impl TryFrom for Network { #[cfg(test)] mod tests { - use super::Network; + use super::{Network, TestnetVersion}; use crate::consensus::encode::{deserialize, serialize}; use crate::p2p::ServiceFlags; #[test] fn serialize_test() { assert_eq!(serialize(&Network::Bitcoin.magic()), &[0xf9, 0xbe, 0xb4, 0xd9]); - assert_eq!(serialize(&Network::Testnet.magic()), &[0x0b, 0x11, 0x09, 0x07]); + assert_eq!( + serialize(&Network::Testnet(TestnetVersion::V3).magic()), + &[0x0b, 0x11, 0x09, 0x07] + ); + assert_eq!( + serialize(&Network::Testnet(TestnetVersion::V4).magic()), + &[0x1c, 0x16, 0x3f, 0x28] + ); assert_eq!(serialize(&Network::Signet.magic()), &[0x0a, 0x03, 0xcf, 0x40]); assert_eq!(serialize(&Network::Regtest.magic()), &[0xfa, 0xbf, 0xb5, 0xda]); assert_eq!(deserialize(&[0xf9, 0xbe, 0xb4, 0xd9]).ok(), Some(Network::Bitcoin.magic())); - assert_eq!(deserialize(&[0x0b, 0x11, 0x09, 0x07]).ok(), Some(Network::Testnet.magic())); + assert_eq!( + deserialize(&[0x0b, 0x11, 0x09, 0x07]).ok(), + Some(Network::Testnet(TestnetVersion::V3).magic()) + ); + assert_eq!( + deserialize(&[0x1c, 0x16, 0x3f, 0x28]).ok(), + Some(Network::Testnet(TestnetVersion::V4).magic()) + ); assert_eq!(deserialize(&[0x0a, 0x03, 0xcf, 0x40]).ok(), Some(Network::Signet.magic())); assert_eq!(deserialize(&[0xfa, 0xbf, 0xb5, 0xda]).ok(), Some(Network::Regtest.magic())); } @@ -330,12 +396,14 @@ mod tests { #[test] fn string_test() { assert_eq!(Network::Bitcoin.to_string(), "bitcoin"); - assert_eq!(Network::Testnet.to_string(), "testnet"); + assert_eq!(Network::Testnet(TestnetVersion::V3).to_string(), "testnet"); + assert_eq!(Network::Testnet(TestnetVersion::V4).to_string(), "testnet4"); assert_eq!(Network::Regtest.to_string(), "regtest"); assert_eq!(Network::Signet.to_string(), "signet"); assert_eq!("bitcoin".parse::().unwrap(), Network::Bitcoin); - assert_eq!("testnet".parse::().unwrap(), Network::Testnet); + assert_eq!("testnet".parse::().unwrap(), Network::Testnet(TestnetVersion::V3)); + assert_eq!("testnet4".parse::().unwrap(), Network::Testnet(TestnetVersion::V4)); assert_eq!("regtest".parse::().unwrap(), Network::Regtest); assert_eq!("signet".parse::().unwrap(), Network::Signet); assert!("fakenet".parse::().is_err()); @@ -386,8 +454,13 @@ mod tests { #[cfg(feature = "serde")] fn serde_roundtrip() { use Network::*; - let tests = - [(Bitcoin, "bitcoin"), (Testnet, "testnet"), (Signet, "signet"), (Regtest, "regtest")]; + let tests = vec![ + (Bitcoin, "bitcoin"), + (Testnet(TestnetVersion::V3), "testnet"), + (Testnet(TestnetVersion::V4), "testnet4"), + (Signet, "signet"), + (Regtest, "regtest"), + ]; for tc in tests { let network = tc.0; @@ -405,7 +478,8 @@ mod tests { fn from_to_core_arg() { let expected_pairs = [ (Network::Bitcoin, "main"), - (Network::Testnet, "test"), + (Network::Testnet(TestnetVersion::V3), "test"), + (Network::Testnet(TestnetVersion::V4), "testnet4"), (Network::Regtest, "regtest"), (Network::Signet, "signet"), ]; diff --git a/bitcoin/src/network/params.rs b/bitcoin/src/network/params.rs index ca93169c6..e17f5bc1c 100644 --- a/bitcoin/src/network/params.rs +++ b/bitcoin/src/network/params.rs @@ -3,7 +3,7 @@ //! Bitcoin consensus parameters. //! //! This module provides a predefined set of parameters for different Bitcoin -//! chains (such as mainnet, testnet). +//! chains (such as mainnet, testnet, testnet4). //! //! # Custom Signets Example //! @@ -73,6 +73,7 @@ use crate::network::Network; #[cfg(doc)] use crate::pow::CompactTarget; use crate::pow::Target; +use crate::TestnetVersion; /// Parameters that influence chain consensus. #[non_exhaustive] @@ -127,8 +128,13 @@ pub struct Params { /// /// [using-statics-or-consts]: pub static MAINNET: Params = Params::MAINNET; -/// The testnet parameters. -pub static TESTNET: Params = Params::TESTNET; +/// The testnet3 parameters. +#[deprecated(since = "0.33.0", note = "Use TESTNET3 instead")] +pub static TESTNET: Params = Params::TESTNET3; +/// The testnet3 parameters. +pub static TESTNET3: Params = Params::TESTNET3; +/// The testnet4 parameters. +pub static TESTNET4: Params = Params::TESTNET4; /// The signet parameters. pub static SIGNET: Params = Params::SIGNET; /// The regtest parameters. @@ -156,9 +162,10 @@ impl Params { no_pow_retargeting: false, }; - /// The testnet parameters. + /// The testnet3 parameters. + #[deprecated(since = "0.33.0", note = "Use TESTNET3 instead")] pub const TESTNET: Params = Params { - network: Network::Testnet, + network: Network::Testnet(TestnetVersion::V3), bip16_time: 1333238400, // Apr 1 2012 bip34_height: BlockHeight::from_u32(21111), // 0000000023b3a96d3484e5abb3755c413e7d41500f8e2a5c3f0dd01299cd8ef8 bip65_height: BlockHeight::from_u32(581885), // 00000000007f6655f22f98e72ed80d8b06dc761d5da09df0fa1dc4be4f861eb6 @@ -173,6 +180,40 @@ impl Params { no_pow_retargeting: false, }; + /// The testnet3 parameters. + pub const TESTNET3: Params = Params { + network: Network::Testnet(TestnetVersion::V3), + bip16_time: 1333238400, // Apr 1 2012 + bip34_height: BlockHeight::from_u32(21111), // 0000000023b3a96d3484e5abb3755c413e7d41500f8e2a5c3f0dd01299cd8ef8 + bip65_height: BlockHeight::from_u32(581885), // 00000000007f6655f22f98e72ed80d8b06dc761d5da09df0fa1dc4be4f861eb6 + bip66_height: BlockHeight::from_u32(330776), // 000000002104c8c45e99a8853285a3b592602a3ccde2b832481da85e9e4ba182 + rule_change_activation_threshold: BlockInterval::from_u32(1512), // 75% + miner_confirmation_window: BlockInterval::from_u32(2016), + pow_limit: Target::MAX_ATTAINABLE_TESTNET, + max_attainable_target: Target::MAX_ATTAINABLE_TESTNET, + pow_target_spacing: 10 * 60, // 10 minutes. + pow_target_timespan: 14 * 24 * 60 * 60, // 2 weeks. + allow_min_difficulty_blocks: true, + no_pow_retargeting: false, + }; + + /// The testnet4 parameters. + pub const TESTNET4: Params = Params { + network: Network::Testnet(TestnetVersion::V4), + bip16_time: 1333238400, // Apr 1 2012 + bip34_height: BlockHeight::from_u32(1), + bip65_height: BlockHeight::from_u32(1), + bip66_height: BlockHeight::from_u32(1), + rule_change_activation_threshold: BlockInterval::from_u32(1512), // 75% + miner_confirmation_window: BlockInterval::from_u32(2016), + pow_limit: Target::MAX_ATTAINABLE_TESTNET, + max_attainable_target: Target::MAX_ATTAINABLE_TESTNET, + pow_target_spacing: 10 * 60, // 10 minutes. + pow_target_timespan: 14 * 24 * 60 * 60, // 2 weeks. + allow_min_difficulty_blocks: true, + no_pow_retargeting: false, + }; + /// The signet parameters. pub const SIGNET: Params = Params { network: Network::Signet, @@ -207,11 +248,12 @@ impl Params { no_pow_retargeting: true, }; - /// Creates parameters set for the given network. /// Creates parameters set for the given network. + /// Creates parameters set for the given network. pub const fn new(network: Network) -> Self { match network { Network::Bitcoin => Params::MAINNET, - Network::Testnet => Params::TESTNET, + Network::Testnet(TestnetVersion::V3) => Params::TESTNET3, + Network::Testnet(TestnetVersion::V4) => Params::TESTNET4, Network::Signet => Params::SIGNET, Network::Regtest => Params::REGTEST, } @@ -244,12 +286,5 @@ impl AsRef for Params { } impl AsRef for Network { - fn as_ref(&self) -> &Params { - match *self { - Network::Bitcoin => &MAINNET, - Network::Testnet => &TESTNET, - Network::Signet => &SIGNET, - Network::Regtest => ®TEST, - } - } + fn as_ref(&self) -> &Params { Self::params(*self) } } diff --git a/bitcoin/src/p2p/mod.rs b/bitcoin/src/p2p/mod.rs index a682ca398..f860a9fea 100644 --- a/bitcoin/src/p2p/mod.rs +++ b/bitcoin/src/p2p/mod.rs @@ -28,7 +28,7 @@ use internals::{debug_from_display, impl_to_hex_from_lower_hex, write_err}; use io::{BufRead, Write}; use crate::consensus::encode::{self, Decodable, Encodable}; -use crate::network::{Network, Params}; +use crate::network::{Network, Params, TestnetVersion}; use crate::prelude::{Borrow, BorrowMut, String, ToOwned}; #[rustfmt::skip] @@ -216,8 +216,13 @@ pub struct Magic([u8; 4]); impl Magic { /// Bitcoin mainnet network magic bytes. pub const BITCOIN: Self = Self([0xF9, 0xBE, 0xB4, 0xD9]); - /// Bitcoin testnet network magic bytes. + /// Bitcoin testnet3 network magic bytes. + #[deprecated(since = "0.33.0", note = "Use TESTNET3 instead")] pub const TESTNET: Self = Self([0x0B, 0x11, 0x09, 0x07]); + /// Bitcoin testnet3 network magic bytes. + pub const TESTNET3: Self = Self([0x0B, 0x11, 0x09, 0x07]); + /// Bitcoin testnet4 network magic bytes. + pub const TESTNET4: Self = Self([0x1c, 0x16, 0x3f, 0x28]); /// Bitcoin signet network magic bytes. pub const SIGNET: Self = Self([0x0A, 0x03, 0xCF, 0x40]); /// Bitcoin regtest network magic bytes. @@ -245,12 +250,12 @@ impl FromStr for Magic { } macro_rules! generate_network_magic_conversion { - ($(Network::$network:ident => Magic::$magic:ident,)*) => { + ($(Network::$network:ident$((TestnetVersion::$testnet_version:ident))? => Magic::$magic:ident,)*) => { impl From for Magic { fn from(network: Network) -> Magic { match network { $( - Network::$network => Magic::$magic, + Network::$network$((TestnetVersion::$testnet_version))? => Magic::$magic, )* } } @@ -262,7 +267,7 @@ macro_rules! generate_network_magic_conversion { fn try_from(magic: Magic) -> Result { match magic { $( - Magic::$magic => Ok(Network::$network), + Magic::$magic => Ok(Network::$network$((TestnetVersion::$testnet_version))?), )* _ => Err(UnknownMagicError(magic)), } @@ -270,10 +275,12 @@ macro_rules! generate_network_magic_conversion { } }; } - +// Generate conversion functions for all known networks. +// `Network -> Magic` and `Magic -> Network` generate_network_magic_conversion! { Network::Bitcoin => Magic::BITCOIN, - Network::Testnet => Magic::TESTNET, + Network::Testnet(TestnetVersion::V3) => Magic::TESTNET3, + Network::Testnet(TestnetVersion::V4) => Magic::TESTNET4, Network::Signet => Magic::SIGNET, Network::Regtest => Magic::REGTEST, } @@ -430,7 +437,8 @@ mod tests { fn magic_from_str() { let known_network_magic_strs = [ ("f9beb4d9", Network::Bitcoin), - ("0b110907", Network::Testnet), + ("0b110907", Network::Testnet(TestnetVersion::V3)), + ("1c163f28", Network::Testnet(TestnetVersion::V4)), ("fabfb5da", Network::Regtest), ("0a03cf40", Network::Signet), ]; diff --git a/bitcoin/tests/psbt-sign-taproot.rs b/bitcoin/tests/psbt-sign-taproot.rs index 6f4563988..3d5a71d77 100644 --- a/bitcoin/tests/psbt-sign-taproot.rs +++ b/bitcoin/tests/psbt-sign-taproot.rs @@ -94,7 +94,7 @@ fn psbt_sign_taproot() { // let keystore = Keystore { mfp: mfp.parse::().unwrap(), - sk: PrivateKey::new(kp.secret_key(), Network::Testnet), + sk: PrivateKey::new(kp.secret_key(), Network::Testnet(bitcoin::TestnetVersion::V3)), }; let _ = psbt_key_path_spend.sign(&keystore, secp); @@ -124,7 +124,7 @@ fn psbt_sign_taproot() { let keystore = Keystore { mfp: mfp.parse::().unwrap(), - sk: PrivateKey::new(kp.secret_key(), Network::Testnet), + sk: PrivateKey::new(kp.secret_key(), Network::Testnet(bitcoin::TestnetVersion::V3)), }; // @@ -195,7 +195,7 @@ fn create_taproot_tree( fn create_p2tr_address(tree: TaprootSpendInfo) -> Address { let output_key = tree.output_key(); - Address::p2tr_tweaked(output_key, Network::Testnet) + Address::p2tr_tweaked(output_key, Network::Testnet(bitcoin::TestnetVersion::V3)) } fn create_psbt_for_taproot_key_path_spend(