fix: Destroy cameras after failed initialization (#1551)

src/camera-manager/manager.ts

@@ -237,14 +237,18 @@ export class CameraManager {
       async ([cameraConfig, engine]) => await engine.createCamera(hass, cameraConfig),
     );
 
+    const destroyCameras = async () => {
+      cameras.forEach((camera) => camera.destroy());
+    };
     const cameraIDs: Set<string> = new Set();
 
     // Do the additions based off the result-order, to ensure the map order is
     // preserved.
-    cameras.forEach((camera) => {
+    for (const camera of cameras) {
       const cameraID = getCameraID(camera.getConfig());
 
       if (!cameraID) {
+        await destroyCameras();
         throw new CameraInitializationError(
           localize('error.no_camera_id'),
           camera.getConfig(),
@@ -252,6 +256,7 @@ export class CameraManager {
       }
 
       if (cameraIDs.has(cameraID)) {
+        await destroyCameras();
         throw new CameraInitializationError(
           localize('error.duplicate_camera_id'),
           camera.getConfig(),
@@ -261,7 +266,7 @@ export class CameraManager {
       // Always ensure the actual ID used in the card is in the configuration itself.
       camera.setID(cameraID);
       cameraIDs.add(cameraID);
-    });
+    }
 
     await this._store.setCameras(cameras);
 

src/scss/message.scss

@@ -57,5 +57,5 @@ div.message div.icon {
 
 .message pre {
   margin-top: 20px;
-  overflow-x: scroll;
+  overflow-x: auto;
 }