robojerk/codeql-action

Fix Unguarded actions library use code scanning error

Browse source 
Must access the env var directly in order to avoid recursive calls to 
`getRequiredEnvParam`.
This commit is contained in:
Andrew Eisenberg 2021-06-02 11:51:21 -07:00
parent 082575fbc3
commit cc0733fd12
3 changed files with 8 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

4
lib/util.js generated
View file

@ -396,7 +396,9 @@ function getRequiredEnvParam(paramName) {
if (value === undefined || value.length === 0) {
throw new Error(`${paramName} environment variable must be set`);
}
core.debug(`${paramName}=${value}`);
if (process.env[EnvVar.RUN_MODE] === Mode.actions) {
core.debug(`${paramName}=${value}`);
}
return value;
}
exports.getRequiredEnvParam = getRequiredEnvParam;

2
lib/util.js.map
View file

File diff suppressed because one or more lines are too long

5
src/util.ts
View file

@ -466,7 +466,10 @@ export function getRequiredEnvParam(paramName: string): string {
if (value === undefined || value.length === 0) {
throw new Error(`${paramName} environment variable must be set`);
}
core.debug(`${paramName}=${value}`);
if (process.env[EnvVar.RUN_MODE] === Mode.actions) {
core.debug(`${paramName}=${value}`);
}
return value;
}