Stop progress bar on message.complete

.github/workflows/releases.yml

@@ -9,17 +9,11 @@ on:
 
 jobs:
   test:
-    outputs:
-      sha: ${{ steps.commit.outputs.sha }}
-      tag: ${{ steps.tag.outputs.tag }}
     runs-on: macos-latest
+
     steps:
       - name: setup repo
         uses: actions/checkout@v3
-      - id: commit
-        run: echo "sha=$(git rev-parse HEAD)" >> "$GITHUB_OUTPUT"
-      - id: tag
-        run: echo "tag=@actions/${{ github.event.inputs.package }}@$(cat packages/${{ github.event.inputs.package }}/package.json | jq -r .version" >> "$GITHUB_OUTPUT"
 
       - name: verify package exists
         run: ls packages/${{ github.event.inputs.package }}
@@ -83,26 +77,4 @@ jobs:
           curl -X POST -H 'Content-type: application/json' --data '{"text":":dance: Successfully published a new version of ${{ github.event.inputs.package }}"}' $SLACK_WEBHOOK
         env:
           SLACK_WEBHOOK: ${{ secrets.SLACK }}
-  tag:
-    name: "Tag ${{ needs.test.outputs.sha }} with ${{ needs.test.outputs.tag }}"
-    runs-on: "macos-latest"
-    needs: [test, publish]
-    steps:
-    - uses: actions/checkout@v3
-    - name: Create tag
-      uses: actions/github-script@v6
-      with:
-        script: |
-          github.rest.git.createRef({
-            owner: context.repo.owner,
-            repo: context.repo.repo,
-            ref: 'refs/tags/${{ needs.test.outputs.tag }}',
-            sha: '${{ needs.test.outputs.sha }}'
-          })
-    - name: notify slack on failure
-      if: failure()
-      run: |
-        curl -X POST -H 'Content-type: application/json' --data '{"text":":pb__failed: Publish of ${{ github.event.inputs.package }} was successful, but failed to tag commit ${{ needs.test.outputs.sha }} with version ${{ needs.test.outputs.tag }}"}' $SLACK_WEBHOOK
-      env:
-        SLACK_WEBHOOK: ${{ secrets.SLACK }}
 

.github/workflows/tag-test.yml

@@ -1,46 +0,0 @@
-name: Tag test
-
-on: push
-
-jobs:
-  test:
-    outputs:
-      sha: ${{ steps.commit.outputs.sha }}
-      tag: ${{ steps.tag.outputs.tag }}
-    runs-on: macos-latest
-    steps:
-      - name: setup repo
-        uses: actions/checkout@v3
-      - id: commit
-        run: echo "sha=$(git rev-parse HEAD)" >> "$GITHUB_OUTPUT"
-      - id: tag
-        run: echo "tag=@actions/http-client@$(cat packages/http-client/package.json | jq -r .version)-test-tag" >> "$GITHUB_OUTPUT"
-  publish:
-    runs-on: macos-latest
-    needs: test
-    steps:
-      - name: skip publish
-        run: echo "skip publish"
-  tag:
-    name: "Tag ${{ needs.test.outputs.sha }} with ${{ needs.test.outputs.tag }}"
-    runs-on: "macos-latest"
-    needs: [test, publish]
-    steps:
-    - uses: actions/checkout@v3
-    - name: Create tag
-      uses: actions/github-script@v6
-      with:
-        script: |
-          github.rest.git.createRef({
-            owner: context.repo.owner,
-            repo: context.repo.repo,
-            ref: 'refs/tags/${{ needs.test.outputs.tag }}',
-            sha: '${{ needs.test.outputs.sha }}'
-          })
-    # - name: notify slack on failure
-    # if: failure()
-    # run: |
-    #   curl -X POST -H 'Content-type: application/json' --data '{"text":":pb__failed: Publish of ${{ github.event.inputs.package }} was successful, but failed to tag commit ${{ needs.test.outputs.sha }} with version ${{ needs.test.outputs.tag }}"}' $SLACK_WEBHOOK
-    # env:
-    #   SLACK_WEBHOOK: ${{ secrets.SLACK }}
-

packages/cache/RELEASES.md

@@ -114,7 +114,4 @@
 - Fix issue with symlink restoration on windows.
 
 ### 3.1.3
-- Fix to prevent from setting MYSYS environement variable globally [#1329](https://github.com/actions/toolkit/pull/1329).
-
-### 3.1.4
-- Fix zstd not being used due to `zstd --version` output change in zstd 1.5.4 release. See [#1353](https://github.com/actions/toolkit/pull/1353).
+- Fix to prevent from setting MYSYS environement variable globally [#1329](https://github.com/actions/toolkit/pull/1329).

packages/cache/package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "@actions/cache",
-  "version": "3.1.4",
+  "version": "3.1.3",
   "lockfileVersion": 2,
   "requires": true,
   "packages": {
     "": {
       "name": "@actions/cache",
-      "version": "3.1.4",
+      "version": "3.1.3",
       "license": "MIT",
       "dependencies": {
         "@actions/core": "^1.10.0",

packages/cache/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@actions/cache",
-  "version": "3.1.4",
+  "version": "3.1.3",
   "preview": true,
   "description": "Actions cache lib",
   "keywords": [

packages/cache/src/internal/cacheUtils.ts

@@ -71,15 +71,11 @@ export async function unlinkFile(filePath: fs.PathLike): Promise<void> {
   return util.promisify(fs.unlink)(filePath)
 }
 
-async function getVersion(
-  app: string,
-  additionalArgs: string[] = []
-): Promise<string> {
+async function getVersion(app: string): Promise<string> {
+  core.debug(`Checking ${app} --version`)
   let versionOutput = ''
-  additionalArgs.push('--version')
-  core.debug(`Checking ${app} ${additionalArgs.join(' ')}`)
   try {
-    await exec.exec(`${app}`, additionalArgs, {
+    await exec.exec(`${app} --version`, [], {
       ignoreReturnCode: true,
       silent: true,
       listeners: {
@@ -98,14 +94,18 @@ async function getVersion(
 
 // Use zstandard if possible to maximize cache performance
 export async function getCompressionMethod(): Promise<CompressionMethod> {
-  const versionOutput = await getVersion('zstd', ['--quiet'])
+  const versionOutput = await getVersion('zstd')
   const version = semver.clean(versionOutput)
-  core.debug(`zstd version: ${version}`)
 
-  if (versionOutput === '') {
+  if (!versionOutput.toLowerCase().includes('zstd command line interface')) {
+    // zstd is not installed
     return CompressionMethod.Gzip
-  } else {
+  } else if (!version || semver.lt(version, 'v1.3.2')) {
+    // zstd is installed but using a version earlier than v1.3.2
+    // v1.3.2 is required to use the `--long` options in zstd
     return CompressionMethod.ZstdWithoutLong
+  } else {
+    return CompressionMethod.Zstd
   }
 }
 

packages/cache/src/internal/downloadUtils.ts

@@ -10,7 +10,7 @@ import * as util from 'util'
 import * as utils from './cacheUtils'
 import {SocketTimeout} from './constants'
 import {DownloadOptions} from '../options'
-import {retryHttpClientResponse} from './requestUtils'
+import {retryHttpClientResponse, sleep} from './requestUtils'
 
 import {AbortController} from '@azure/abort-controller'
 
@@ -161,6 +161,28 @@ export class DownloadProgress {
   }
 }
 
+async function displayDownloadProgress(message: any, startTime: number): Promise<void> {
+  const socket = message.socket
+  while(!message.complete) {
+    const byteRead = socket.bytesRead
+    const totalBytes = 100000
+    const percentage = (100 * (byteRead / totalBytes)).toFixed(
+      1
+    )
+    const elapsedTime = Date.now() - startTime
+    const downloadSpeed = (
+      byteRead /
+      (1024 * 1024) /
+      (elapsedTime / 1000)
+    ).toFixed(1)
+
+    core.info(
+      `Received ${byteRead} of ${totalBytes} (${percentage}%), ${downloadSpeed} MBs/sec`
+    )
+    sleep(100)
+  }
+}
+
 /**
  * Download the cache using the Actions toolkit http-client
  *
@@ -171,6 +193,7 @@ export async function downloadCacheHttpClient(
   archiveLocation: string,
   archivePath: string
 ): Promise<void> {
+  const startTime = Date.now()
   const writeStream = fs.createWriteStream(archivePath)
   const httpClient = new HttpClient('actions/cache')
   const downloadResponse = await retryHttpClientResponse(
@@ -184,6 +207,8 @@ export async function downloadCacheHttpClient(
     core.debug(`Aborting download, socket timed out after ${SocketTimeout} ms`)
   })
 
+  await displayDownloadProgress(downloadResponse.message, startTime)
+
   await pipeResponseToStream(downloadResponse, writeStream)
 
   // Validate download size.

packages/cache/src/internal/requestUtils.ts

@@ -33,7 +33,7 @@ export function isRetryableStatusCode(statusCode?: number): boolean {
   return retryableStatusCodes.includes(statusCode)
 }
 
-async function sleep(milliseconds: number): Promise<void> {
+export async function sleep(milliseconds: number): Promise<void> {
   return new Promise(resolve => setTimeout(resolve, milliseconds))
 }
 

packages/http-client/__tests__/proxy.test.ts

@@ -145,44 +145,6 @@ describe('proxy', () => {
     expect(bypass).toBeFalsy()
   })
 
-  it('checkBypass returns true if host with subdomain in no_proxy', () => {
-    process.env['no_proxy'] = 'myserver.com'
-    const bypass = pm.checkBypass(new URL('https://sub.myserver.com'))
-    expect(bypass).toBeTruthy()
-  })
-
-  it('checkBypass returns false if no_proxy is subdomain', () => {
-    process.env['no_proxy'] = 'myserver.com'
-    const bypass = pm.checkBypass(new URL('https://myserver.com.evil.org'))
-    expect(bypass).toBeFalsy()
-  })
-
-  it('checkBypass returns false if no_proxy is part of domain', () => {
-    process.env['no_proxy'] = 'myserver.com'
-    const bypass = pm.checkBypass(new URL('https://evilmyserver.com'))
-    expect(bypass).toBeFalsy()
-  })
-
-  // Do not strip leading dots as per https://github.com/actions/runner/blob/97195bad5870e2ad0915ebfef1616083aacf5818/docs/adrs/0263-proxy-support.md
-  it('checkBypass returns false if host with leading dot in no_proxy', () => {
-    process.env['no_proxy'] = '.myserver.com'
-    const bypass = pm.checkBypass(new URL('https://myserver.com'))
-    expect(bypass).toBeFalsy()
-  })
-
-  it('checkBypass returns true if host with subdomain in no_proxy defined with leading "."', () => {
-    process.env['no_proxy'] = '.myserver.com'
-    const bypass = pm.checkBypass(new URL('https://sub.myserver.com'))
-    expect(bypass).toBeTruthy()
-  })
-
-  // Do not match wildcard ("*") as per https://github.com/actions/runner/blob/97195bad5870e2ad0915ebfef1616083aacf5818/docs/adrs/0263-proxy-support.md
-  it('checkBypass returns true if no_proxy is "*"', () => {
-    process.env['no_proxy'] = '*'
-    const bypass = pm.checkBypass(new URL('https://anything.whatsoever.com'))
-    expect(bypass).toBeFalsy()
-  })
-
   it('HttpClient does basic http get request through proxy', async () => {
     process.env['http_proxy'] = _proxyUrl
     const httpClient = new httpm.HttpClient()

packages/http-client/src/proxy.ts

@@ -51,15 +51,7 @@ export function checkBypass(reqUrl: URL): boolean {
     .split(',')
     .map(x => x.trim().toUpperCase())
     .filter(x => x)) {
-    if (
-      upperReqHosts.some(
-        x =>
-          x === upperNoProxyItem ||
-          x.endsWith(`.${upperNoProxyItem}`) ||
-          (upperNoProxyItem.startsWith('.') &&
-            x.endsWith(`${upperNoProxyItem}`))
-      )
-    ) {
+    if (upperReqHosts.some(x => x === upperNoProxyItem)) {
       return true
     }
   }