X-Git-Url: http://git.euphorik.ch/index.cgi?a=blobdiff_plain;f=backend%2Fsrc%2Fminecraft_controller.rs;h=8f67d200111f1de90e971bdf3ff524488ff23e2e;hb=34c1bf4bcb51473466ee9398cb63f0049ddc06a0;hp=a644c4db95bf7e5b616daf044bf2523ea2db1528;hpb=e4d360ba5fcd9ecd5717215403ed60ec64cd4b7b;p=minecraft_web.git

diff --git a/backend/src/minecraft_controller.rs b/backend/src/minecraft_controller.rs
index a644c4d..8f67d20 100644
--- a/backend/src/minecraft_controller.rs
+++ b/backend/src/minecraft_controller.rs
@@ -88,8 +88,13 @@ fn get_status_from_rcon(rcon_password: &str) -> StatusFromRcon {
                         match client.send_command("list".to_string()) {
                             Ok(resp) =>
                                 match resp.body.find(':') {
-                                    Some(i) => resp.body[i + 1..resp.body.len() - 1].split(',').map(|nick| nick.trim().to_string()).collect(),
-                                    None => Vec::new()
+                                    Some(i) if i < resp.body.len() -1 =>
+                                        resp.body[i + 1..resp.body.len()]
+                                            .split(',')
+                                            .map(|nick| nick.trim().to_string())
+                                            .filter(|nick| !nick.is_empty())
+                                            .collect(),
+                                    _ => Vec::new()
                                 },
                             Err(_e) => {
                                 println!("Error from 'list' command");
@@ -127,12 +132,10 @@ fn get_last_backup_datetime(backup_path: &str) -> Option<SystemTime> {
 pub fn get_minecraft_executable_information(world_path: &str, backup_path: &str, rcon_password: &str) -> Option<MinecraftExe> {
     let mut system = sysinfo::System::new_all();
     system.refresh_system();
-    let processes = system.get_process_by_name(MINECRAFT_PROCESS_NAME);
+    let mut processes = system.processes_by_name(MINECRAFT_PROCESS_NAME);
 
     // TODO: find the correct process by checking the correct jar name in parameters.
-    if processes.len() >= 1 {
-        let process = processes.first().unwrap();
-
+    if let Some(process) = processes.next() {
         let world_size = match fs_extra::dir::get_size(world_path) { Ok(l) => l, Err(_) => 0u64 };
 
         let status_from_rcon = get_status_from_rcon(rcon_password);
@@ -140,7 +143,7 @@ pub fn get_minecraft_executable_information(world_path: &str, backup_path: &str,
         Some(
             MinecraftExe {
                 memory: process.memory(),
-                load_average_5min: system.get_load_average().five / system.get_processors().len() as f64 * 100.,
+                load_average_5min: system.load_average().five / system.processors().len() as f64 * 100.,
                 uptime: std::time::SystemTime::now().duration_since(std::time::UNIX_EPOCH).unwrap().as_secs() - process.start_time(),
                 world_size,
                 active_players: status_from_rcon.players,