Remove deprecated lightning::util::string re-exports

These were deprecated in 0.1 in favor of the `lightning::types`
re-export of the `lightning-types` crate.

Fixes #3251.

Author: TheBlueMatt

fuzz/src/invoice_request_deser.rs

@@ -24,8 +24,8 @@ use lightning::offers::parse::Bolt12SemanticError;
 use lightning::sign::EntropySource;
 use lightning::types::features::BlindedHopFeatures;
 use lightning::types::payment::{PaymentHash, PaymentSecret};
+use lightning::types::string::UntrustedString;
 use lightning::util::ser::Writeable;
-use lightning::util::string::UntrustedString;
 
 #[inline]
 pub fn do_test<Out: test_logger::Output>(data: &[u8], _out: Out) {

lightning-persister/src/fs_store.rs

@@ -1,8 +1,8 @@
 //! Objects related to [`FilesystemStore`] live here.
 use crate::utils::{check_namespace_key_validity, is_valid_kvstore_str};
 
+use lightning::types::string::PrintableString;
 use lightning::util::persist::{KVStore, MigratableKVStore};
-use lightning::util::string::PrintableString;
 
 use std::collections::HashMap;
 use std::fs;

lightning-persister/src/utils.rs

@@ -1,5 +1,5 @@
+use lightning::types::string::PrintableString;
 use lightning::util::persist::{KVSTORE_NAMESPACE_KEY_ALPHABET, KVSTORE_NAMESPACE_KEY_MAX_LEN};
-use lightning::util::string::PrintableString;
 
 pub(crate) fn is_valid_kvstore_str(key: &str) -> bool {
 	key.len() <= KVSTORE_NAMESPACE_KEY_MAX_LEN

lightning/src/events/mod.rs

@@ -35,12 +35,12 @@ use crate::routing::router::{BlindedTail, Path, RouteHop, RouteParameters};
 use crate::sign::SpendableOutputDescriptor;
 use crate::types::features::ChannelTypeFeatures;
 use crate::types::payment::{PaymentHash, PaymentPreimage, PaymentSecret};
+use crate::types::string::UntrustedString;
 use crate::util::errors::APIError;
 use crate::util::ser::{
 	BigSize, FixedLengthReader, MaybeReadable, Readable, RequiredWrapper, UpgradableRequired,
 	WithoutLength, Writeable, Writer,
 };
-use crate::util::string::UntrustedString;
 
 use crate::io;
 use crate::sync::Arc;
@@ -315,7 +315,7 @@ pub enum ClosureReason {
 		/// a security vulnerability in the terminal emulator or the logging subsystem.
 		/// To be safe, use `Display` on `UntrustedString`
 		///
-		/// [`UntrustedString`]: crate::util::string::UntrustedString
+		/// [`UntrustedString`]: crate::types::string::UntrustedString
 		peer_msg: UntrustedString,
 	},
 	/// Closure generated from [`ChannelManager::force_close_channel`], called by the user.

lightning/src/ln/channelmanager.rs

@@ -119,6 +119,7 @@ use crate::types::features::{
 	Bolt12InvoiceFeatures, ChannelFeatures, ChannelTypeFeatures, InitFeatures, NodeFeatures,
 };
 use crate::types::payment::{PaymentHash, PaymentPreimage, PaymentSecret};
+use crate::types::string::UntrustedString;
 use crate::util::config::{ChannelConfig, ChannelConfigOverrides, ChannelConfigUpdate, UserConfig};
 use crate::util::errors::APIError;
 use crate::util::logger::{Level, Logger, WithContext};
@@ -127,7 +128,6 @@ use crate::util::ser::{
 	BigSize, FixedLengthReader, LengthReadable, MaybeReadable, Readable, ReadableArgs, VecWriter,
 	Writeable, Writer,
 };
-use crate::util::string::UntrustedString;
 use crate::util::wakers::{Future, Notifier};
 
 #[cfg(all(test, async_payments))]

lightning/src/ln/peer_handler.rs

@@ -45,10 +45,10 @@ use crate::onion_message::packet::OnionMessageContents;
 use crate::routing::gossip::{NodeAlias, NodeId};
 use crate::sign::{NodeSigner, Recipient};
 use crate::types::features::{InitFeatures, NodeFeatures};
+use crate::types::string::PrintableString;
 use crate::util::atomic_counter::AtomicCounter;
 use crate::util::logger::{Level, Logger, WithContext};
 use crate::util::ser::{VecWriter, Writeable, Writer};
-use crate::util::string::PrintableString;
 
 #[allow(unused_imports)]
 use crate::prelude::*;

lightning/src/offers/invoice.rs

@@ -147,11 +147,11 @@ use crate::offers::refund::{
 use crate::offers::signer::{self, Metadata};
 use crate::types::features::{Bolt12InvoiceFeatures, InvoiceRequestFeatures, OfferFeatures};
 use crate::types::payment::PaymentHash;
+use crate::types::string::PrintableString;
 use crate::util::ser::{
 	CursorReadable, HighZeroBytesDroppedBigSize, Iterable, LengthLimitedRead, LengthReadable,
 	WithoutLength, Writeable, Writer,
 };
-use crate::util::string::PrintableString;
 use bitcoin::address::Address;
 use bitcoin::constants::ChainHash;
 use bitcoin::secp256k1::schnorr::Signature;
@@ -1802,8 +1802,8 @@ mod tests {
 	use crate::offers::test_utils::*;
 	use crate::prelude::*;
 	use crate::types::features::{Bolt12InvoiceFeatures, InvoiceRequestFeatures, OfferFeatures};
+	use crate::types::string::PrintableString;
 	use crate::util::ser::{BigSize, Iterable, Writeable};
-	use crate::util::string::PrintableString;
 	#[cfg(not(c_bindings))]
 	use {crate::offers::offer::OfferBuilder, crate::offers::refund::RefundBuilder};
 	#[cfg(c_bindings)]

lightning/src/offers/invoice_error.rs

@@ -13,8 +13,8 @@ use crate::io;
 use crate::ln::msgs::DecodeError;
 use crate::offers::merkle::SignError;
 use crate::offers::parse::Bolt12SemanticError;
+use crate::types::string::UntrustedString;
 use crate::util::ser::{HighZeroBytesDroppedBigSize, Readable, WithoutLength, Writeable, Writer};
-use crate::util::string::UntrustedString;
 
 #[allow(unused_imports)]
 use crate::prelude::*;
@@ -123,10 +123,10 @@ mod tests {
 	use super::{ErroneousField, InvoiceError};
 
 	use crate::ln::msgs::DecodeError;
+	use crate::types::string::UntrustedString;
 	use crate::util::ser::{
 		HighZeroBytesDroppedBigSize, Readable, VecWriter, WithoutLength, Writeable,
 	};
-	use crate::util::string::UntrustedString;
 
 	#[test]
 	fn parses_invoice_error_without_erroneous_field() {

lightning/src/offers/invoice_request.rs

@@ -85,11 +85,11 @@ use crate::offers::signer::{Metadata, MetadataMaterial};
 use crate::onion_message::dns_resolution::HumanReadableName;
 use crate::types::features::InvoiceRequestFeatures;
 use crate::types::payment::PaymentHash;
+use crate::types::string::{PrintableString, UntrustedString};
 use crate::util::ser::{
 	CursorReadable, HighZeroBytesDroppedBigSize, LengthLimitedRead, LengthReadable, Readable,
 	WithoutLength, Writeable, Writer,
 };
-use crate::util::string::{PrintableString, UntrustedString};
 use bitcoin::constants::ChainHash;
 use bitcoin::network::Network;
 use bitcoin::secp256k1::schnorr::Signature;
@@ -1474,8 +1474,8 @@ mod tests {
 	use crate::offers::payer::PayerTlvStreamRef;
 	use crate::offers::test_utils::*;
 	use crate::types::features::{InvoiceRequestFeatures, OfferFeatures};
+	use crate::types::string::{PrintableString, UntrustedString};
 	use crate::util::ser::{BigSize, Readable, Writeable};
-	use crate::util::string::{PrintableString, UntrustedString};
 	use bitcoin::constants::ChainHash;
 	use bitcoin::network::Network;
 	use bitcoin::secp256k1::{self, Keypair, Secp256k1, SecretKey};

lightning/src/offers/offer.rs

@@ -87,11 +87,11 @@ use crate::offers::nonce::Nonce;
 use crate::offers::parse::{Bech32Encode, Bolt12ParseError, Bolt12SemanticError, ParsedMessage};
 use crate::offers::signer::{self, Metadata, MetadataMaterial};
 use crate::types::features::OfferFeatures;
+use crate::types::string::PrintableString;
 use crate::util::ser::{
 	CursorReadable, HighZeroBytesDroppedBigSize, LengthLimitedRead, LengthReadable, Readable,
 	WithoutLength, Writeable, Writer,
 };
-use crate::util::string::PrintableString;
 use bitcoin::constants::ChainHash;
 use bitcoin::network::Network;
 use bitcoin::secp256k1::{self, Keypair, PublicKey, Secp256k1};
@@ -646,7 +646,7 @@ macro_rules! offer_accessors { ($self: ident, $contents: expr) => {
 
 	/// A complete description of the purpose of the payment. Intended to be displayed to the user
 	/// but with the caveat that it has not been verified in any way.
-	pub fn description(&$self) -> Option<$crate::util::string::PrintableString> {
+	pub fn description(&$self) -> Option<$crate::types::string::PrintableString> {
 		$contents.description()
 	}
 
@@ -664,7 +664,7 @@ macro_rules! offer_accessors { ($self: ident, $contents: expr) => {
 
 	/// The issuer of the offer, possibly beginning with `user@domain` or `domain`. Intended to be
 	/// displayed to the user but with the caveat that it has not been verified in any way.
-	pub fn issuer(&$self) -> Option<$crate::util::string::PrintableString> {
+	pub fn issuer(&$self) -> Option<$crate::types::string::PrintableString> {
 		$contents.issuer()
 	}
 
@@ -1349,8 +1349,8 @@ mod tests {
 	use crate::offers::parse::{Bolt12ParseError, Bolt12SemanticError};
 	use crate::offers::test_utils::*;
 	use crate::types::features::OfferFeatures;
+	use crate::types::string::PrintableString;
 	use crate::util::ser::{BigSize, Writeable};
-	use crate::util::string::PrintableString;
 	use bitcoin::constants::ChainHash;
 	use bitcoin::network::Network;
 	use bitcoin::secp256k1::Secp256k1;