Skip to content

Commit 6505e97

Browse files
authored
Merge pull request #420 from jcreedcmu/jcreed/fix-release-asset-search
Only look for codeql.zip assets
2 parents 572e74e + a6fc0d5 commit 6505e97

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

extensions/ql-vscode/src/distribution.ts

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -327,8 +327,13 @@ class ExtensionSpecificDistributionManager {
327327

328328
private async getLatestRelease(): Promise<Release> {
329329
const release = await this.createReleasesApiConsumer().getLatestRelease(this._versionConstraint, this._config.includePrerelease);
330-
if (release.assets.length !== 1) {
331-
throw new Error("Release had an unexpected number of assets");
330+
// FIXME: Look for platform-specific codeql distribution if available
331+
release.assets = release.assets.filter(asset => asset.name === 'codeql.zip');
332+
if (release.assets.length === 0) {
333+
throw new Error("Release had no asset named codeql.zip");
334+
}
335+
else if (release.assets.length > 1) {
336+
throw new Error("Release had more than one asset named codeql.zip");
332337
}
333338
return release;
334339
}

0 commit comments

Comments
 (0)