From 670eda8d916403684e2c4cc0ca77d0d5310e48ae Mon Sep 17 00:00:00 2001 From: Jonathan Miller Date: Sat, 6 Jun 2026 16:07:23 -0500 Subject: [PATCH] =?UTF-8?q?fix:=20download=20key=20lost=20on=20update=20?= =?UTF-8?q?=E2=80=94=20stale=20URL=20in=20cleanupStaleUpdateSites?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The cleanup method used the old /raw/branch/main/updates.xml URL as its keep target, but migrateUpdateServerUrls had already rewritten all URLs to /updates.xml. This caused the method to find no match and delete all MokoWaaS update sites — including the one with the download key. Fixed by updating the hardcoded URL to match the current manifest format: /MokoConsulting/MokoWaaS/updates.xml Authored-by: Moko Consulting Co-Authored-By: Claude Opus 4.6 (1M context) --- CHANGELOG.md | 3 +++ source/script.php | 2 +- 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 07a027d5..a332fcde 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -22,6 +22,9 @@ ## [Unreleased] +### Fixed +- Download key lost on update: cleanupStaleUpdateSites used old /raw/branch/main/ URL format, deleting the manifest-registered update site that held the key + ## [02.35.00] - 2026-06-06 ### Added diff --git a/source/script.php b/source/script.php index f9739a46..e13c52d6 100644 --- a/source/script.php +++ b/source/script.php @@ -601,7 +601,7 @@ class Pkg_MokowaasInstallerScript try { $db = Factory::getDbo(); - $dynamicUrl = 'https://git.mokoconsulting.tech/MokoConsulting/MokoWaaS/raw/branch/main/updates.xml'; + $dynamicUrl = 'https://git.mokoconsulting.tech/MokoConsulting/MokoWaaS/updates.xml'; // Find all MokoWaaS update sites $query = $db->getQuery(true)