aquatic_ws: save peer even if AnnounceRequest.bytes_left is None

aquatic_ws/src/lib/common.rs

@@ -37,11 +37,11 @@ impl PeerStatus {
     #[inline]
     pub fn from_event_and_bytes_left(
         event: AnnounceEvent,
-        bytes_left: usize
+        opt_bytes_left: Option<usize>
     ) -> Self {
         if let AnnounceEvent::Stopped = event {
             Self::Stopped
-        } else if bytes_left == 0 {
+        } else if let Some(0) = opt_bytes_left {
             Self::Seeding
         } else {
             Self::Leeching

aquatic_ws/src/lib/handler.rs

@@ -109,11 +109,9 @@ pub fn handle_announce_requests(
             }
         }
 
-        // FIXME: correct to only update when bytes_left is Some?
-        if let Some(bytes_left) = request.bytes_left {
         let peer_status = PeerStatus::from_event_and_bytes_left(
             request.event,
-                bytes_left
+            request.bytes_left
         );
 
         let peer = Peer {
@@ -147,7 +145,6 @@ pub fn handle_announce_requests(
             },
             _ => {}
         }
-        }
 
         // If peer sent offers, send them on to random peers
         if let Some(offers) = request.offers {

aquatic_ws/src/lib/protocol/mod.rs

@@ -103,7 +103,8 @@ pub struct AnnounceRequestOffer {
 pub struct AnnounceRequest {
     pub info_hash: InfoHash,
     pub peer_id: PeerId,
-    /// Just called "left" in protocol
+    /// Just called "left" in protocol. Is set to None in some cases, such as
+    /// when opening a magnet link
     #[serde(rename = "left")]
     pub bytes_left: Option<usize>,
     /// Can be empty. Then, default is "update"