Merge pull request #2672 from github/mbg/start-proxy/include-type-in-…

…urls-output
github · Jan 8, 2025 · fb65b6c · fb65b6c
2 parents 3407610 + b25558b
commit fb65b6c
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 4 deletions.
diff --git a/lib/start-proxy-action.js b/lib/start-proxy-action.js
diff --git a/lib/start-proxy-action.js.map b/lib/start-proxy-action.js.map
diff --git a/src/start-proxy-action.ts b/src/start-proxy-action.ts
@@ -174,7 +174,10 @@ async function startProxy(
 
     const registry_urls = config.all_credentials
       .filter((credential) => credential.url !== undefined)
-      .map((credential) => credential.url);
+      .map((credential) => ({
+        type: credential.type,
+        url: credential.url,
+      }));
     core.setOutput("proxy_urls", JSON.stringify(registry_urls));
   } catch (error) {
     core.setFailed(`start-proxy action failed: ${util.getErrorMessage(error)}`);

diff --git a/start-proxy/action.yml b/start-proxy/action.yml
@@ -24,7 +24,7 @@ outputs:
   proxy_ca_certificate:
     description: The proxy's internal CA certificate in PEM format
   proxy_urls:
-    description: The URLs of the configured registries, as a JSON array.
+    description: A stringified JSON array of objects containing the types and URLs of the configured registries.
 runs:
   using: node20
   main: "../lib/start-proxy-action.js"