Require Clone + Sync + Send + 'static for event and connection loggers

.. we should consider dropping `Deref` and instead just commiting to
store a `Arc<L>` everwhere, as it gets tedious to maintain.

However, this is barely scraping by and the least invasive change here.

Author: tnull

src/connection.rs

@@ -1,4 +1,4 @@
-use crate::logger::{log_error, log_info, FilesystemLogger, Logger};
+use crate::logger::{log_error, log_info, Logger};
 use crate::types::PeerManager;
 use crate::Error;
 
@@ -7,13 +7,16 @@ use lightning::ln::msgs::SocketAddress;
 use bitcoin::secp256k1::PublicKey;
 
 use std::net::ToSocketAddrs;
+use std::ops::Deref;
 use std::sync::Arc;
 use std::time::Duration;
 
-pub(crate) async fn connect_peer_if_necessary(
-	node_id: PublicKey, addr: SocketAddress, peer_manager: Arc<PeerManager>,
-	logger: Arc<FilesystemLogger>,
-) -> Result<(), Error> {
+pub(crate) async fn connect_peer_if_necessary<L: Deref + Clone + Sync + Send>(
+	node_id: PublicKey, addr: SocketAddress, peer_manager: Arc<PeerManager>, logger: L,
+) -> Result<(), Error>
+where
+	L::Target: Logger,
+{
 	for (pman_node_id, _pman_addr) in peer_manager.get_peer_node_ids() {
 		if node_id == pman_node_id {
 			return Ok(());
@@ -23,10 +26,12 @@ pub(crate) async fn connect_peer_if_necessary(
 	do_connect_peer(node_id, addr, peer_manager, logger).await
 }
 
-pub(crate) async fn do_connect_peer(
-	node_id: PublicKey, addr: SocketAddress, peer_manager: Arc<PeerManager>,
-	logger: Arc<FilesystemLogger>,
-) -> Result<(), Error> {
+pub(crate) async fn do_connect_peer<L: Deref + Clone + Sync + Send>(
+	node_id: PublicKey, addr: SocketAddress, peer_manager: Arc<PeerManager>, logger: L,
+) -> Result<(), Error>
+where
+	L::Target: Logger,
+{
 	log_info!(logger, "Connecting to peer: {}@{}", node_id, addr);
 
 	let socket_addr = addr

src/event.rs

@@ -280,7 +280,7 @@ impl Future for EventFuture {
 	}
 }
 
-pub(crate) struct EventHandler<L: Deref>
+pub(crate) struct EventHandler<L: Deref + Clone + Sync + Send + 'static>
 where
 	L::Target: Logger,
 {
@@ -296,7 +296,7 @@ where
 	config: Arc<Config>,
 }
 
-impl<L: Deref> EventHandler<L>
+impl<L: Deref + Clone + Sync + Send + 'static> EventHandler<L>
 where
 	L::Target: Logger,
 {