f: Rename handle_message_received -> message_received

Since the function is called when any message from any handler is
received, and is not dependent on any specific message, rename the
function to be more generic and improve the documentation.

Author: shaavan

lightning-net-tokio/src/lib.rs

@@ -787,7 +787,7 @@ mod tests {
 		fn get_chain_hashes(&self) -> Option<Vec<ChainHash>> {
 			Some(vec![ChainHash::using_genesis_block(Network::Testnet)])
 		}
-		fn handle_message_received(&self) {}
+		fn message_received(&self) {}
 	}
 	impl MessageSendEventsProvider for MsgHandler {
 		fn get_and_clear_pending_msg_events(&self) -> Vec<MessageSendEvent> {

lightning/src/ln/channelmanager.rs

@@ -10747,7 +10747,7 @@ where
 			 msg.channel_id.clone())), *counterparty_node_id);
 	}
 
-	fn handle_message_received(&self) {
+	fn message_received(&self) {
 		for (payment_id, retryable_invoice_request) in self
 			.pending_outbound_payments
 			.release_invoice_requests_awaiting_invoice()

lightning/src/ln/msgs.rs

@@ -1606,9 +1606,10 @@ pub trait ChannelMessageHandler : MessageSendEventsProvider {
 	/// connecting to peers.
 	fn get_chain_hashes(&self) -> Option<Vec<ChainHash>>;
 
-	/// Handles behavior which is triggered when any message from any peer for any handler
-	/// is received.
-	fn handle_message_received(&self);
+	/// Indicates that a message was received from any peer for any handler.
+	/// Called before the message is passed to the appropriate handler.
+	/// Useful for indicating that a network connection is active.
+	fn message_received(&self);
 }
 
 /// A trait to describe an object which can receive routing messages.

lightning/src/ln/peer_handler.rs

@@ -388,7 +388,7 @@ impl ChannelMessageHandler for ErroringMessageHandler {
 		ErroringMessageHandler::push_error(self, their_node_id, msg.channel_id);
 	}
 
-	fn handle_message_received(&self) {}
+	fn message_received(&self) {}
 }
 
 impl Deref for ErroringMessageHandler {
@@ -1752,7 +1752,7 @@ impl<Descriptor: SocketDescriptor, CM: Deref, RM: Deref, OM: Deref, L: Deref, CM
 			log_trace!(logger, "Received message {:?} from {}", message, log_pubkey!(their_node_id));
 		}
 
-		self.message_handler.chan_handler.handle_message_received();
+		self.message_handler.chan_handler.message_received();
 
 		let mut should_forward = None;
 

lightning/src/util/test_utils.rs

@@ -937,7 +937,7 @@ impl msgs::ChannelMessageHandler for TestChannelMessageHandler {
 		self.received_msg(wire::Message::TxAbort(msg.clone()));
 	}
 
-	fn handle_message_received(&self) {}
+	fn message_received(&self) {}
 }
 
 impl events::MessageSendEventsProvider for TestChannelMessageHandler {