refactor: [#1524] command/query separation

The returned value is not needed anymore. Secondary action (increase
metrics) is done in the event listeners.

Author: josecelano

packages/axum-http-tracker-server/src/environment.rs

@@ -25,12 +25,12 @@ pub struct Environment<S> {
 
 impl<S> Environment<S> {
     /// Add a torrent to the tracker
-    pub async fn add_torrent_peer(&self, info_hash: &InfoHash, peer: &peer::Peer) -> bool {
+    pub async fn add_torrent_peer(&self, info_hash: &InfoHash, peer: &peer::Peer) {
         self.container
             .tracker_core_container
             .in_memory_torrent_repository
             .handle_announcement(info_hash, peer, None)
-            .await
+            .await;
     }
 }
 

packages/axum-rest-tracker-api-server/src/environment.rs

@@ -33,12 +33,12 @@ where
     S: std::fmt::Debug + std::fmt::Display,
 {
     /// Add a torrent to the tracker
-    pub async fn add_torrent_peer(&self, info_hash: &InfoHash, peer: &peer::Peer) -> bool {
+    pub async fn add_torrent_peer(&self, info_hash: &InfoHash, peer: &peer::Peer) {
         self.container
             .tracker_core_container
             .in_memory_torrent_repository
             .handle_announcement(info_hash, peer, None)
-            .await
+            .await;
     }
 }