mrva/vscode-codeql
1
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

Switch ALLOW_HTTP_SETTING for allowHttp()

Browse Source
This commit is contained in:
Robert
2023-07-21 14:53:49 +01:00
parent f0318b0c84
commit a79b71cff6
2 changed files with 3 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
extensions/ql-vscode/src/config.ts
View File

@@ -649,10 +649,7 @@ export function isCodespacesTemplate() {
const DATABASE_DOWNLOAD_SETTING = new Setting("databaseDownload", ROOT_SETTING);
export const ALLOW_HTTP_SETTING = new Setting(
"allowHttp",
DATABASE_DOWNLOAD_SETTING,
);
const ALLOW_HTTP_SETTING = new Setting("allowHttp", DATABASE_DOWNLOAD_SETTING);
export function allowHttp(): boolean {
return ALLOW_HTTP_SETTING.getValue<boolean>() || false;

4
extensions/ql-vscode/src/databases/database-fetcher.ts
View File

@@ -30,7 +30,7 @@ import {
} from "../common/github-url-identifier-helper";
import { Credentials } from "../common/authentication";
import { AppCommandManager } from "../common/commands";
import { ALLOW_HTTP_SETTING } from "../config";
import { allowHttp } from "../config";
import { showAndLogInformationMessage } from "../common/logging";
/**
@@ -371,7 +371,7 @@ function validateUrl(databaseUrl: string) {
throw new Error(`Invalid url: ${databaseUrl}`);
}
if (!ALLOW_HTTP_SETTING.getValue() && uri.scheme !== "https") {
if (!allowHttp() && uri.scheme !== "https") {
throw new Error("Must use https for downloading a database.");
}
}