From fb1d550793937c53f912f12d098ca3493610e70d Mon Sep 17 00:00:00 2001
From: Will Greenberg <willg@eff.org>
Date: Wed, 16 Apr 2025 10:57:33 -0700
Subject: [PATCH] when deleting all, close the current recording first

---
 bin/src/diag.rs | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/bin/src/diag.rs b/bin/src/diag.rs
index e545902..2294fa2 100644
--- a/bin/src/diag.rs
+++ b/bin/src/diag.rs
@@ -257,13 +257,6 @@ pub async fn delete_all_recordings(
     if state.debug_mode {
         return Err((StatusCode::FORBIDDEN, "server is in debug mode".to_string()));
     }
-    let mut qmdl_store = state.qmdl_store_lock.write().await;
-    qmdl_store.delete_all_entries().await.map_err(|e| {
-        (
-            StatusCode::INTERNAL_SERVER_ERROR,
-            format!("couldn't delete all recordings: {}", e),
-        )
-    })?;
     state
         .diag_device_ctrl_sender
         .send(DiagDeviceCtrlMessage::StopRecording)
@@ -274,6 +267,13 @@ pub async fn delete_all_recordings(
                 format!("couldn't send stop recording message: {}", e),
             )
         })?;
+    let mut qmdl_store = state.qmdl_store_lock.write().await;
+    qmdl_store.delete_all_entries().await.map_err(|e| {
+        (
+            StatusCode::INTERNAL_SERVER_ERROR,
+            format!("couldn't delete all recordings: {}", e),
+        )
+    })?;
     state
         .ui_update_sender
         .send(display::DisplayState::Paused)