refactor: remove need for clone

bytebeamio · Mar 27, 2024 · aa46feb · aa46feb
1 parent e42b296
commit aa46feb
Showing 1 changed file with 9 additions and 9 deletions.
diff --git a/uplink/src/collector/downloader.rs b/uplink/src/collector/downloader.rs
@@ -229,6 +229,15 @@ impl FileDownloader {
             }
         }
 
+        state.current.meta.verify_checksum()?;
+        info!("Firmware downloaded successfully");
+
+        let mut action = state.current.action;
+        action.payload = serde_json::to_string(&state.current.meta)?;
+        let status = ActionResponse::done(&self.action_id, "Downloaded", Some(action))
+            .set_sequence(self.sequence());
+        self.bridge_tx.send_action_response(status).await;
+
         Ok(())
     }
 
@@ -262,15 +271,6 @@ impl FileDownloader {
             }
         }
 
-        state.current.meta.verify_checksum()?;
-        info!("Firmware downloaded successfully");
-
-        let mut action = state.current.action.clone();
-        action.payload = serde_json::to_string(&state.current.meta)?;
-        let status = ActionResponse::done(&self.action_id, "Downloaded", Some(action))
-            .set_sequence(self.sequence());
-        self.bridge_tx.send_action_response(status).await;
-
         Ok(())
     }