fix: localhost resolution RFC6761 (#70)

* spike: investigate localhost resolution RFC6761

* bump curl to 7.85.0

* fix

* pull latest curl-for-windows & bump to 7.86.0

.github/workflows/build-and-release.yaml

@@ -49,7 +49,7 @@ jobs:
           - macos-13-xlarge
           - ubuntu-22.04
         libcurl-release:
-          - 7.79.1
+          - 7.86.0
         node-libcurl-cpp-std:
           - c++17
         node:
@@ -108,7 +108,7 @@ jobs:
           - macos-13-xlarge
           - ubuntu-22.04
         libcurl-release:
-          - 7.79.1
+          - 7.86.0
         node:
           - 20.16.0
         electron-version:

.github/workflows/build-lint-test.yaml

@@ -43,7 +43,7 @@ jobs:
           - macos-13-xlarge
           - ubuntu-22.04
         libcurl-release:
-          - 7.79.1
+          - 7.86.0
         node-libcurl-cpp-std:
           - c++17
         node:
@@ -53,7 +53,7 @@ jobs:
           - os: ubuntu-latest
             node: 20.16.0
             node-libcurl-cpp-std: c++17
-            libcurl-release: 7.79.1
+            libcurl-release: 7.86.0
             run-lint-and-tsc: true
 
     env:
@@ -141,7 +141,7 @@ jobs:
           - macos-13-xlarge
           - ubuntu-22.04
         libcurl-release:
-          - 7.79.1
+          - 7.86.0
         node:
           - 20.16.0
         electron-version:

test/curl/localhost.spec.ts

@@ -0,0 +1,75 @@
+import 'should'
+import { app, host, port, server } from '../helper/server'
+import { Curl } from '../../lib'
+
+let curl: Curl
+const urlLocalhost = `http://localhost:${port}/`
+const urlTestLocalhost = `http://test.localhost:${port}/`
+
+describe('DNS Resolution', () => {
+  before((done) => {
+    server.listen(port, host, () => done())
+
+    app.get('/', (req, res) => {
+      res.send({ message: 'resolved', ip: req.ip })
+    })
+  })
+
+  after((done) => {
+    server.close()
+    app._router.stack.pop()
+    done()
+  })
+
+  beforeEach(() => {
+    curl = new Curl()
+  })
+
+  afterEach(() => {
+    curl.close()
+  })
+
+  it('should resolve localhost to 127.0.0.1', (done) => {
+    curl.setOpt('URL', urlLocalhost)
+
+    curl.on('end', (status, data) => {
+      if (status !== 200) {
+        throw Error(`Invalid status code: ${status}`)
+      }
+
+      const result = JSON.parse(data as string)
+      result.message.should.be.equal('resolved')
+      result.ip.should.be.equal('::1')
+
+      done()
+    })
+
+    curl.on('error', done)
+    curl.perform()
+  })
+
+  it('should resolve test.localhost to 127.0.0.1', (done) => {
+    // skip this test if windows because it does not support *.localhost on hosts file?
+    // https://stackoverflow.com/questions/138162/wildcards-in-a-windows-hosts-file/4166967#4166967
+    // if (process.platform === 'win32') {
+    //   done()
+    //   return
+    // }
+    curl.setOpt('URL', urlTestLocalhost)
+
+    curl.on('end', (status, data) => {
+      if (status !== 200) {
+        throw Error(`Invalid status code: ${status}`)
+      }
+
+      const result = JSON.parse(data as string)
+      result.message.should.be.equal('resolved')
+      result.ip.should.be.equal('::1')
+
+      done()
+    })
+
+    curl.on('error', done)
+    curl.perform()
+  })
+})