Merge pull request #77 from feross/gh-actions

Add GitHub Actions CI

.github/workflows/test.yml

@@ -0,0 +1,24 @@
+name: Node CI
+
+on:
+  pull_request:
+    branches:
+    - master
+
+jobs:
+  test:
+    strategy:
+      matrix:
+        os: [macos-latest, ubuntu-latest, windows-latest]
+        node: [10.0.0, 12.0.0, 14.0.0, 16.0.0]
+    runs-on: ${{ matrix.os }}
+    steps:
+    - uses: actions/checkout@v2
+    - name: Use Node.js ${{ matrix.node }}
+      uses: actions/setup-node@v2
+      with:
+        node-version: ${{ matrix.node }}
+    - name: Install dependencies
+      run: npm install
+    - name: Run tests
+      run: npm test

.gitignore

@@ -1,3 +1 @@
-node_modules/
-yarn.lock
-package-lock.json
+/node_modules/

.npmrc

@@ -0,0 +1 @@
+package-lock=false

package.json

@@ -24,6 +24,12 @@
     "string-to-stream": "^3.0.0",
     "tape": "^5.0.0"
   },
+  "engines": {
+    "node": ">=10"
+  },
+  "files": [
+    "index.js"
+  ],
   "homepage": "https://github.com/feross/simple-get",
   "keywords": [
     "request",

test/auth.js

@@ -25,3 +25,49 @@ test('basic auth', function (t) {
     })
   })
 })
+
+test('basic auth + host', function (t) {
+  t.plan(5)
+
+  const server = http.createServer(function (req, res) {
+    t.equal(req.headers.authorization, 'Basic Zm9vOmJhcg==')
+    res.statusCode = 200
+    res.end('response')
+  })
+
+  server.listen(0, function () {
+    const port = server.address().port
+    get({ auth: 'foo:bar', host: 'localhost', port }, function (err, res) {
+      t.error(err)
+      t.equal(res.statusCode, 200)
+      concat(res, function (err, data) {
+        t.error(err)
+        t.equal(data.toString(), 'response')
+        server.close()
+      })
+    })
+  })
+})
+
+test('basic auth + hostname', function (t) {
+  t.plan(5)
+
+  const server = http.createServer(function (req, res) {
+    t.equal(req.headers.authorization, 'Basic Zm9vOmJhcg==')
+    res.statusCode = 200
+    res.end('response')
+  })
+
+  server.listen(0, function () {
+    const port = server.address().port
+    get({ auth: 'foo:bar', hostname: 'localhost', port }, function (err, res) {
+      t.error(err)
+      t.equal(res.statusCode, 200)
+      concat(res, function (err, data) {
+        t.error(err)
+        t.equal(data.toString(), 'response')
+        server.close()
+      })
+    })
+  })
+})

test/basic.js

@@ -170,3 +170,26 @@ test('rewrite POST redirects to GET', function (t) {
     })
   })
 })
+
+test('simple get hostname + url', function (t) {
+  t.plan(5)
+
+  const server = http.createServer(function (req, res) {
+    t.equal(req.url, '/path')
+    res.statusCode = 200
+    res.end('response')
+  })
+
+  server.listen(0, function () {
+    const port = server.address().port
+    get({ host: 'localhost', port, url: '/path' }, function (err, res) {
+      t.error(err)
+      t.equal(res.statusCode, 200)
+      concat(res, function (err, data) {
+        t.error(err)
+        t.equal(data.toString(), 'response')
+        server.close()
+      })
+    })
+  })
+})

test/redirect.js

@@ -326,3 +326,35 @@ test('redirect should clear explicitly specified `Host` (note uppercase) header'
     })
   })
 })
+
+test('follow redirects without "url" option', function (t) {
+  t.plan(15)
+
+  let num = 0
+  const server = http.createServer(function (req, res) {
+    t.equal(req.url, '/' + num, 'visited /' + num)
+
+    if (num < 10) {
+      num += 1
+      res.statusCode = 301
+      res.setHeader('Location', '/' + num)
+      res.end()
+    } else {
+      res.statusCode = 200
+      res.end('response')
+    }
+  })
+
+  server.listen(0, function () {
+    const port = server.address().port
+    get({ hostname: 'localhost', port, path: '/0' }, function (err, res) {
+      t.error(err)
+      t.equal(res.statusCode, 200)
+      concat(res, function (err, data) {
+        t.error(err)
+        t.equal(data.toString(), 'response')
+        server.close()
+      })
+    })
+  })
+})