LotusGuild/gandalf
3
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Cancel in-flight diagnostic poll when user selects a new port
Lint / Python (flake8) (push) Successful in 45s
Details
Lint / JS (eslint) (push) Successful in 10s
Details
Security / Python Security (bandit) (push) Successful in 52s
Details
Test / Python Tests (pytest) (push) Successful in 1m2s
Details
Lint / Notify on failure (push) Has been skipped
Details
Lint / Deploy (push) Successful in 2s
Details

Browse Source 
Previously switching ports while a diagnostic was running left the
setInterval timer active, causing the result to be written into the
old (now detached) DOM elements and never shown to the user.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Jared Vititoe
2026-05-11 23:26:53 -04:00
parent a1a3a52dd8
commit 2d6dcd782f
1 changed files with 1 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
templates/inspector.html
+1
View File
@@ -218,6 +218,7 @@ let _apiData = null;
function selectPort(el) { function selectPort(el) {
const swName = el.dataset.switch; const swName = el.dataset.switch;
const idx = parseInt(el.dataset.portIdx, 10); const idx = parseInt(el.dataset.portIdx, 10);
if (_diagPollTimer) { clearInterval(_diagPollTimer); _diagPollTimer = null; }
document.querySelectorAll('.switch-port-block.selected') document.querySelectorAll('.switch-port-block.selected')
.forEach(e => e.classList.remove('selected')); .forEach(e => e.classList.remove('selected'));
el.classList.add('selected'); el.classList.add('selected');