comments + todo

TODO.md

@@ -0,0 +1,2 @@
+# Todo List
+- Redo the entire download process. Currently it is very very clunky, and the implementation is terrible. It essentially acts as a second client and it would be much better if this was either implemented as part of the original client (server-sided client), or if the entire download flow was redone both client and server side.

src/server/client.rs

@@ -138,6 +138,7 @@ impl Client {
                     // I think I will simply keep a counter around that will
                     // track what the next mod is per client.
                     // TODO: Clean this up. It also needs to be moved out of the client code IMO
+
                     let id = self.read_raw(1).await?[0] as usize;
                     debug!("HandleDownload connection for client id: {}", id);
 
@@ -151,6 +152,11 @@ impl Client {
 
                         *next_id += 1;
 
+                        if *next_id >= config.mods.len() {
+                            // I think this is where the connection should be closed, instead of after
+                            // just 1 mod.
+                        }
+
                         bmod.0.clone()
                     };