Merge pull request lightningdevkit#9 from lightsparkdev/waterson/dont-panic-gossip

Don't panic if we can't sign a gossip message.

Author: waterson

lightning/src/ln/channelmanager.rs

@@ -3110,11 +3110,17 @@ where
 			fee_proportional_millionths: chan.context.get_fee_proportional_millionths(),
 			excess_data: Vec::new(),
 		};
+
+		// TODO(waterson): disabled this so that things work; not sure how to upstream.
 		// Panic on failure to signal LDK should be restarted to retry signing the `ChannelUpdate`.
 		// If we returned an error and the `node_signer` cannot provide a signature for whatever
 		// reason`, we wouldn't be able to receive inbound payments through the corresponding
 		// channel.
-		let sig = self.node_signer.sign_gossip_message(msgs::UnsignedGossipMessage::ChannelUpdate(&unsigned)).unwrap();
+		let sig = self.node_signer.sign_gossip_message(msgs::UnsignedGossipMessage::ChannelUpdate(&unsigned))
+			.map_err(|_| LightningError {
+				err: "Could not sign channel announcement".to_owned(),
+				action: msgs::ErrorAction::IgnoreError,
+			})?;
 
 		Ok(msgs::ChannelUpdate {
 			signature: sig,