Make Endpoint::close not owning

iroh-net/src/endpoint.rs

@@ -847,20 +847,18 @@ impl Endpoint {
     ///
     /// Returns an error if closing the magic socket failed.
     /// TODO: Document error cases.
-    pub async fn close(self, error_code: VarInt, reason: &[u8]) -> Result<()> {
+    pub async fn close(&self, error_code: VarInt, reason: &[u8]) -> Result<()> {
         let Endpoint {
             msock,
             endpoint,
             cancel_token,
             ..
-        } = self;
+        } = &self;
         cancel_token.cancel();
         tracing::debug!("Closing connections");
         endpoint.close(error_code, reason);
         endpoint.wait_idle().await;
-        // In case this is the last clone of `Endpoint`, dropping the `quinn::Endpoint` will
-        // make it more likely that the underlying socket is not polled by quinn anymore after this
-        drop(endpoint);
+
         tracing::debug!("Connections closed");
 
         msock.close().await?;