From 700a0ab4f0243188bdb14e1f1859dedc95d0d6d1 Mon Sep 17 00:00:00 2001
From: cclohmar <claus.lohmar@lohmar.co.uk>
Date: Thu, 6 Aug 2026 19:02:21 +0100
Subject: [PATCH] =?UTF-8?q?fix:=20use=20element=20IDs=20for=20auth=20check?=
 =?UTF-8?q?boxes=20=E2=80=94=20reliable=20checked=20state?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 public/app.js | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/public/app.js b/public/app.js
index 5de28c1..51c6b4c 100644
--- a/public/app.js
+++ b/public/app.js
@@ -269,7 +269,7 @@ async function renderCameraCards() {
                     <label>Username <input type="text" value="${escAttr(config?.auth?.master?.username || '')}" data-auth="master-user"></label>
                     <label>Password <input type="password" placeholder="new password" data-auth="master-pass"></label>
                     <div class="row" style="margin-top:8px">
-                        <label>Require login on port 8080 <input type="checkbox" ${config?.auth?.enabled ? 'checked' : ''} data-auth="auth-enabled" style="width:auto"></label>
+                        <label>Require login on port 8080 <input type="checkbox" id="auth-enable" ${config?.auth?.enabled ? 'checked' : ''} style="width:auto"></label>
                     </div>
                 </div>
                 <div style="flex:1;min-width:200px">
@@ -277,7 +277,7 @@ async function renderCameraCards() {
                     <label>Username <input type="text" value="${escAttr(config?.auth?.viewer?.username || '')}" data-auth="viewer-user"></label>
                     <label>Password <input type="password" placeholder="new password" data-auth="viewer-pass"></label>
                     <div class="row" style="margin-top:8px">
-                        <label>Enable local view access <input type="checkbox" ${config?.auth?.viewer?.enabled ? 'checked' : ''} data-auth="viewer-enabled" style="width:auto"></label>
+                        <label>Enable local view access <input type="checkbox" id="viewer-enable" ${config?.auth?.viewer?.enabled ? 'checked' : ''} style="width:auto"></label>
                     </div>
                 </div>
             </div>
@@ -420,12 +420,12 @@ document.getElementById('btn-save').addEventListener('click', async () => {
         cfg.cameras = updatedCameras;
 
         // Include auth settings from the management card.
-        const authEnabled = document.querySelector('[data-auth="auth-enabled"]')?.checked || false;
+        const authEnabled = document.getElementById('auth-enable')?.checked || false;
         const masterUser = document.querySelector('[data-auth="master-user"]')?.value || '';
         const masterPass = document.querySelector('[data-auth="master-pass"]')?.value || '';
         const viewerUser = document.querySelector('[data-auth="viewer-user"]')?.value || '';
         const viewerPass = document.querySelector('[data-auth="viewer-pass"]')?.value || '';
-        const viewerEnabled = document.querySelector('[data-auth="viewer-enabled"]')?.checked || false;
+        const viewerEnabled = document.getElementById('viewer-enable')?.checked || false;
 
         cfg.auth = cfg.auth || {};
         cfg.auth.enabled = authEnabled;