From 6028be92efc66fb9f3a334ea49e6603a22350f89 Mon Sep 17 00:00:00 2001
From: Pierre HUBERT <pierre.git@communiquons.org>
Date: Tue, 27 Aug 2024 22:35:07 +0200
Subject: [PATCH] Anticipate relay ID collision

---
 central_backend/src/server/web_api/relays_controller.rs | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/central_backend/src/server/web_api/relays_controller.rs b/central_backend/src/server/web_api/relays_controller.rs
index 63484ee..639eb3f 100644
--- a/central_backend/src/server/web_api/relays_controller.rs
+++ b/central_backend/src/server/web_api/relays_controller.rs
@@ -39,6 +39,15 @@ pub async fn create(actor: WebEnergyActor, req: web::Json<CreateDeviceRelayReque
         return Ok(HttpResponse::BadRequest().json("Too many relays for the target device!"));
     }
 
+    if actor
+        .send(energy_actor::GetSingleRelay(req.relay.id))
+        .await?
+        .is_some()
+    {
+        log::error!("Invalid relay create query: A relay with the same ID already exists!");
+        return Ok(HttpResponse::BadRequest().json("A relay with the same ID already exists!"));
+    }
+
     // Create the device
     actor
         .send(energy_actor::CreateDeviceRelay(