robojerk/codeql-action

Merge pull request #2740 from github/revert-2724-marcogario/skip_proxy

Browse source 
Revert "start-proxy: Skip proxy setup if no credentials are available"
This commit is contained in:
Marco Gario 2025-01-29 11:03:48 +01:00 • committed by GitHub
commit 14b9c0ec59
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
3 changed files with 1 additions and 11 deletions
Download patch file Download diff file Expand all files Collapse all files

4
lib/start-proxy-action.js generated
View file

@ -99,10 +99,6 @@ async function runWrapper() {
core.saveState("proxy-log-file", proxyLogFilePath); core.saveState("proxy-log-file", proxyLogFilePath);
// Get the configuration options // Get the configuration options
const credentials = (0, start_proxy_1.getCredentials)(logger, actionsUtil.getOptionalInput("registry_secrets"), actionsUtil.getOptionalInput("registries_credentials"), actionsUtil.getOptionalInput("language")); const credentials = (0, start_proxy_1.getCredentials)(logger, actionsUtil.getOptionalInput("registry_secrets"), actionsUtil.getOptionalInput("registries_credentials"), actionsUtil.getOptionalInput("language"));
if (credentials.length === 0) {
logger.info("No credentials found, skipping proxy setup.");
return;
}
logger.info(`Credentials loaded for the following registries:\n ${credentials logger.info(`Credentials loaded for the following registries:\n ${credentials
.map((c) => credentialToStr(c)) .map((c) => credentialToStr(c))
.join("\n")}`); .join("\n")}`);

2
lib/start-proxy-action.js.map
View file

File diff suppressed because one or more lines are too long

6
src/start-proxy-action.ts
View file

@ -99,12 +99,6 @@ async function runWrapper() {
actionsUtil.getOptionalInput("registries_credentials"), actionsUtil.getOptionalInput("registries_credentials"),
actionsUtil.getOptionalInput("language"), actionsUtil.getOptionalInput("language"),
); );
if (credentials.length === 0) {
logger.info("No credentials found, skipping proxy setup.");
return;
}
logger.info( logger.info(
`Credentials loaded for the following registries:\n ${credentials `Credentials loaded for the following registries:\n ${credentials
.map((c) => credentialToStr(c)) .map((c) => credentialToStr(c))