Remove minimum_fee_penalty

Author: sourabhmarathe

lightning/src/routing/network_graph.rs

@@ -260,8 +260,6 @@ pub struct ChannelInfo {
 	/// Everything else is useful only for sending out for initial routing sync.
 	/// Not stored if contains excess data to prevent DoS.
 	pub announcement_message: Option<msgs::ChannelAnnouncement>,
-	/// The minimum fee the user is willing to pay to avoid using a channel
-	pub minimum_fee_penalty: Option<u64>,
 }
 
 impl std::fmt::Display for ChannelInfo {
@@ -278,8 +276,7 @@ impl_writeable!(ChannelInfo, 0, {
 	one_to_two,
 	node_two,
 	two_to_one,
-	announcement_message,
-	minimum_fee_penalty
+	announcement_message
 });
 
 /// Fees for routing via a given channel or a node
@@ -439,7 +436,7 @@ pub trait ChannelScorer {
 	/// the metadata they are tracking (eg metrics for reliability, uptime, floppiness, etc.)
 	type Metadata;
 	/// Returns penalty fee for a given channel ID based on user's channel metadata
-	fn calculate_minimum_fee_penalty(&self, _channel_id: u64) -> Option<u64> { None }
+	fn calculate_minimum_fee_penalty(&self, _channel_id: u64) -> u64 { 0 }
 	/// Optional: allows user to count PaymentSent events for channels
 	fn score_payment_success(&self, _route: Vec<Vec<RouteHop>>, _metadata: Vec<Self::Metadata>) {}
 	/// Optional: allows user to count PaymentFailed events for channels
@@ -587,7 +584,6 @@ impl NetworkGraph {
 				node_two: msg.contents.node_id_2.clone(),
 				two_to_one: None,
 				announcement_message: if should_relay { Some(msg.clone()) } else { None },
-				minimum_fee_penalty: None,
 			};
 
 		match self.channels.entry(msg.contents.short_channel_id) {