Update proxy.test.ts

.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/http-client/__tests__/proxy.test.ts

@@ -237,6 +237,31 @@ describe('proxy', () => {
     expect(_proxyConnects).toHaveLength(0)
   })
 
+  it('HttpClient bypasses proxy for loopback addresses (localhost, ::1, 127.*)', async () => {
+    // setup a server listening on localhost:8091
+    const server = http.createServer((request, response) => {
+      response.writeHead(200)
+      request.pipe(response)
+    })
+    server.listen(8091)
+    try {
+      process.env['http_proxy'] = _proxyUrl
+      const httpClient = new httpm.HttpClient()
+      let res = await httpClient.get('http://localhost:8091')
+      expect(res.message.statusCode).toBe(200)
+      res = await httpClient.get('http://127.0.0.1:8091')
+      expect(res.message.statusCode).toBe(200)
+
+      // no support for ipv6 for now
+      expect(httpClient.get('http://[::1]:8091')).rejects.toThrow()
+
+      // proxy at _proxyUrl was ignored
+      expect(_proxyConnects).toEqual([])
+    } finally {
+      server.close()
+    }
+  })
+
   it('proxyAuth not set in tunnel agent when authentication is not provided', async () => {
     process.env['https_proxy'] = 'http://127.0.0.1:8080'
     const httpClient = new httpm.HttpClient()

packages/http-client/src/proxy.ts

@@ -25,6 +25,11 @@ export function checkBypass(reqUrl: URL): boolean {
     return false
   }
 
+  const reqHost = reqUrl.hostname
+  if (isLoopbackAddress(reqHost)) {
+    return true
+  }
+
   const noProxy = process.env['no_proxy'] || process.env['NO_PROXY'] || ''
   if (!noProxy) {
     return false
@@ -66,3 +71,13 @@ export function checkBypass(reqUrl: URL): boolean {
 
   return false
 }
+
+function isLoopbackAddress(host: string): boolean {
+  const hostLower = host.toLowerCase()
+  return (
+    hostLower === 'localhost' ||
+    hostLower.startsWith('127.') ||
+    hostLower.startsWith('[::1]') ||
+    hostLower.startsWith('[0:0:0:0:0:0:0:1]')
+  )
+}