robojerk/codeql-action

Store generated config file in temporary directory

Browse source 
This prevents it from becoming part of the analysis.
This commit is contained in:
Henry Mercer 2024-02-13 15:59:45 +00:00
parent c79c360e02
commit 191f59f435
3 changed files with 5 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

4
lib/config-utils.js generated
View file

@ -462,13 +462,13 @@ function dbLocationOrDefault(dbLocation, tempDir) {
*/ */
async function initConfig(inputs) { async function initConfig(inputs) {
let config; let config;
const { logger, workspacePath } = inputs; const { logger, tempDir } = inputs;
// if configInput is set, it takes precedence over configFile // if configInput is set, it takes precedence over configFile
if (inputs.configInput) { if (inputs.configInput) {
if (inputs.configFile) { if (inputs.configFile) {
logger.warning(`Both a config file and config input were provided. Ignoring config file.`); logger.warning(`Both a config file and config input were provided. Ignoring config file.`);
} }
inputs.configFile = path.resolve(workspacePath, "user-config-from-action.yml"); inputs.configFile = path.resolve(tempDir, "user-config-from-action.yml");
fs.writeFileSync(inputs.configFile, inputs.configInput); fs.writeFileSync(inputs.configFile, inputs.configInput);
logger.debug(`Using config from action input: ${inputs.configFile}`); logger.debug(`Using config from action input: ${inputs.configFile}`);
} }

2
lib/config-utils.js.map
View file

File diff suppressed because one or more lines are too long

7
src/config-utils.ts
View file

@ -813,7 +813,7 @@ function dbLocationOrDefault(
export async function initConfig(inputs: InitConfigInputs): Promise<Config> { export async function initConfig(inputs: InitConfigInputs): Promise<Config> {
let config: Config; let config: Config;
const { logger, workspacePath } = inputs; const { logger, tempDir } = inputs;
// if configInput is set, it takes precedence over configFile // if configInput is set, it takes precedence over configFile
if (inputs.configInput) { if (inputs.configInput) {
@ -822,10 +822,7 @@ export async function initConfig(inputs: InitConfigInputs): Promise<Config> {
`Both a config file and config input were provided. Ignoring config file.`, `Both a config file and config input were provided. Ignoring config file.`,
); );
} }
inputs.configFile = path.resolve( inputs.configFile = path.resolve(tempDir, "user-config-from-action.yml");
workspacePath,
"user-config-from-action.yml",
);
fs.writeFileSync(inputs.configFile, inputs.configInput); fs.writeFileSync(inputs.configFile, inputs.configInput);
logger.debug(`Using config from action input: ${inputs.configFile}`); logger.debug(`Using config from action input: ${inputs.configFile}`);
} }