Skip to content

Commit

Permalink
Don't dump the whole super verbos e correlated URL
Browse files Browse the repository at this point in the history
  • Loading branch information
grahamc committed Mar 13, 2024
1 parent 76946ea commit 17b8547
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 11 deletions.
7 changes: 4 additions & 3 deletions dist/index.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion dist/index.js.map

Large diffs are not rendered by default.

15 changes: 8 additions & 7 deletions src/main.ts
Original file line number Diff line number Diff line change
Expand Up @@ -87,12 +87,6 @@ class IdsToolbox {

fetchUrl.pathname += `/${this.architectureFetchSuffix}`;

fetchUrl.searchParams.set("ci", "github");
fetchUrl.searchParams.set(
"correlation",
JSON.stringify(correlation.identify()),
);

return fetchUrl;
}

Expand Down Expand Up @@ -153,7 +147,14 @@ class IdsToolbox {
async fetch(): Promise<string> {
actions_core.info(`Fetching from ${this.getUrl()}`);

const versionCheckup = await gotClient.head(this.getUrl());
const correlatedUrl = this.getUrl();
correlatedUrl.searchParams.set("ci", "github");
correlatedUrl.searchParams.set(
"correlation",
JSON.stringify(correlation.identify()),
);

const versionCheckup = await gotClient.head(correlatedUrl);
if (versionCheckup.headers.etag) {
const v = versionCheckup.headers.etag;

Expand Down

0 comments on commit 17b8547

Please sign in to comment.