Handle case when both label and hash are missing

microsoft · Oct 8, 2024 · bd696be · bd696be
1 parent bc04919
commit bd696be
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/api/script/utils/acquisition.ts b/api/script/utils/acquisition.ts
@@ -59,8 +59,11 @@ function getUpdatePackage(packageHistory: Package[], request: UpdateCheckRequest
     // Note that older client plugin versions do not send the release label. If the
     // label is supplied, we use label comparison, since developers can release the
     // same update twice. Otherwise, we fall back to hash comparison.
+    // If request is missing both label and hash we take the latest package
+    // as we cannot determine which one the client is running
     foundRequestPackageInHistory =
       foundRequestPackageInHistory ||
+      (!request.label && !request.packageHash) ||
       (request.label && packageEntry.label === request.label) ||
       (!request.label && packageEntry.packageHash === request.packageHash);
     if (packageEntry.isDisabled || (ignoreRolloutPackages && isUnfinishedRollout(packageEntry.rollout))) {