From 71937f6455a0f2ff523645aeb023fec05990cd0c Mon Sep 17 00:00:00 2001 From: Riccardo Casatta Date: Wed, 24 Apr 2024 14:47:46 +0200 Subject: [PATCH 1/2] network_parameters: change bip21 schema liquidtestnet->liquidnetwork the schema doesn't need to convey the network, indeed for bitcoin we have only `bitcoin:` as schema for every network (mainnet, testnet, signet, regtest) This is indeed a breaking change and the liquidtesnet schema is used in the wild, however, since it's breaking only for testnet networks, it makes sense to change it. --- docs/source/examples/get_networks.json | 4 ++-- src/network_parameters.cpp | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/docs/source/examples/get_networks.json b/docs/source/examples/get_networks.json index 8938b66af..949a60de9 100644 --- a/docs/source/examples/get_networks.json +++ b/docs/source/examples/get_networks.json @@ -220,7 +220,7 @@ "asset_registry_onion_url": "http://lhquhzzpzg5tyymcqep24fynpzzqqg3m3rlh7ascnw5cpqsro35bfxyd.onion/testnet/", "asset_registry_url": "https://assets-testnet.blockstream.info/", "bech32_prefix": "tex", - "bip21_prefix": "liquidtestnet", + "bip21_prefix": "liquidnetwork", "blech32_prefix": "tlq", "blinded_prefix": 23, "blob_server_onion_url": "", @@ -480,7 +480,7 @@ "asset_registry_onion_url": "http://lhquhzzpzg5tyymcqep24fynpzzqqg3m3rlh7ascnw5cpqsro35bfxyd.onion/testnet/", "asset_registry_url": "https://assets-testnet.blockstream.info/", "bech32_prefix": "tex", - "bip21_prefix": "liquidtestnet", + "bip21_prefix": "liquidnetwork", "blech32_prefix": "tlq", "blinded_prefix": 23, "blob_server_onion_url": "", diff --git a/src/network_parameters.cpp b/src/network_parameters.cpp index 4a4c1e911..4c46e22cf 100644 --- a/src/network_parameters.cpp +++ b/src/network_parameters.cpp @@ -291,7 +291,7 @@ static std::map> registered_network { "asset_registry_onion_url", "http://lhquhzzpzg5tyymcqep24fynpzzqqg3m3rlh7ascnw5cpqsro35bfxyd.onion/testnet/" }, { "asset_registry_url", "https://assets-testnet.blockstream.info/" }, { "bech32_prefix", "tex" }, - { "bip21_prefix", "liquidtestnet" }, + { "bip21_prefix", "liquidnetwork" }, { "blech32_prefix", "tlq" }, { "blinded_prefix", 23u }, { "blob_server_url", std::string() }, @@ -602,7 +602,7 @@ static std::map> registered_network { "asset_registry_onion_url", "http://lhquhzzpzg5tyymcqep24fynpzzqqg3m3rlh7ascnw5cpqsro35bfxyd.onion/testnet/" }, { "asset_registry_url", "https://assets-testnet.blockstream.info/" }, { "bech32_prefix", "tex" }, - { "bip21_prefix", "liquidtestnet" }, + { "bip21_prefix", "liquidnetwork" }, { "blech32_prefix", "tlq" }, { "blinded_prefix", 23u }, { "blob_server_url", std::string() }, From 381f03cd4a7792fccf64be80d0e9924a343c6e89 Mon Sep 17 00:00:00 2001 From: Riccardo Casatta Date: Wed, 24 Apr 2024 19:17:38 +0200 Subject: [PATCH 2/2] docs: in CHANGELOG mention bip21 schema change for liquid testnet --- CHANGELOG.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index 58e426ef0..08788860a 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -15,6 +15,8 @@ require two-factor authentication. - Build: refactor scripts that build dependencies, add an example dockerfile for android build in bookworm-based image +- Changed bip21 schema for liquid testnet networks, from `liquidtestnet` to + `liquidnetwork`. ### Fixed