f - more fixes

Author: jkczyz

lightning/src/routing/gossip.rs

@@ -925,9 +925,8 @@ impl_writeable_tlv_based!(NodeAnnouncementInfo, {
 
 /// A user-defined name for a node, which may be used when displaying the node in a graph.
 ///
-/// Node aliases provide a potential avenue for injection attacks. See [security considerations].
-///
-/// [security considerations]: https://github.com/lightning/bolts/blob/master/07-routing-gossip.md#security-considerations-for-node-aliases
+/// Since node aliases are provided by third parties, they are a potential avenue for injection
+/// attacks. Care must be taken when processing.
 #[derive(Clone, Debug, PartialEq)]
 pub struct NodeAlias(pub [u8; 32]);
 
@@ -944,6 +943,7 @@ impl fmt::Display for NodeAlias {
 			},
 			Err(_) => {
 				bytes.iter().map(|b| *b as char)
+					// Display printable ASCII characters
 					.map(|c| if c >= '\x20' && c <= '\x7e' { c } else { control_symbol })
 					.collect::<String>()
 			},