Use UntrustedString for displaying errors in PeerManager

We currently just print "with non-ASCII error message"
to log when we see non-ASCII chars, but should instead
use our fancy UntrustedString wrapper to display the
untrusted string and ignore control chars.

Author: vladimirfomene

lightning/src/ln/peer_handler.rs

@@ -31,6 +31,7 @@ use crate::onion_message::{CustomOnionMessageContents, CustomOnionMessageHandler
 use crate::routing::gossip::{NetworkGraph, P2PGossipSync, NodeId, NodeAlias};
 use crate::util::atomic_counter::AtomicCounter;
 use crate::util::logger::Logger;
+use crate::util::string::UntrustedString;
 
 use crate::prelude::*;
 use crate::io;
@@ -1547,7 +1548,7 @@ impl<Descriptor: SocketDescriptor, CM: Deref, RM: Deref, OM: Deref, L: Deref, CM
 				if data_is_printable {
 					log_debug!(self.logger, "Got Err message from {}: {}", log_pubkey!(their_node_id), msg.data);
 				} else {
-					log_debug!(self.logger, "Got Err message from {} with non-ASCII error message", log_pubkey!(their_node_id));
+					log_debug!(self.logger, "Got Err message from {}: {}", log_pubkey!(their_node_id), UntrustedString(msg.data.clone()));
 				}
 				self.message_handler.chan_handler.handle_error(&their_node_id, &msg);
 				if msg.channel_id == [0; 32] {
@@ -1566,7 +1567,7 @@ impl<Descriptor: SocketDescriptor, CM: Deref, RM: Deref, OM: Deref, L: Deref, CM
 				if data_is_printable {
 					log_debug!(self.logger, "Got warning message from {}: {}", log_pubkey!(their_node_id), msg.data);
 				} else {
-					log_debug!(self.logger, "Got warning message from {} with non-ASCII error message", log_pubkey!(their_node_id));
+					log_debug!(self.logger, "Got warning message from {}: {}", log_pubkey!(their_node_id), UntrustedString(msg.data));
 				}
 			},