Add the possibility to set a custom domain

Set a custom domain by setting the DOMAIN environment variable.

Author: PatrikElfstrom

README.md

@@ -43,6 +43,7 @@ serve ~/myproj
 - `sudo` may be necessary.
 - If a static path is not provided the current directory content will be served.
 - You can change the **port** setting the `PORT` environmental variable: `PORT=4433 serve ~/myproj`. Specifying port number will also prevent http to https redirect.
+- You can change the **domain** setting the `DOMAIN` environmental variable: `DOMAIN=192.168.0.2 serve ~/myproj`.
 
 ### Binaries
 If you don't have Node.js installed just use a packaged version! Download it from the [release page](https://github.com/daquinoaldo/https-localhost/releases).

certs.js

@@ -138,7 +138,7 @@ async function generate(appDataPath = CERT_PATH, customDomain = undefined) {
 }
 
 async function getCerts(customDomain = undefined) {
-  const domain = customDomain || "localhost"
+  const domain = process.env.DOMAIN || customDomain || "localhost"
   const certPath = process.env.CERT_PATH || CERT_PATH
   // check for updates if running as executable
   /* istanbul ignore if: cannot test pkg */

index.js

@@ -14,7 +14,7 @@ const getCerts = require(path.resolve(__dirname, "certs.js")).getCerts
 /* CONFIGURE THE SERVER */
 
 // SSL certificate
-const createServer = (domain = "localhost") => {
+const createServer = (domain = process.env.DOMAIN || "localhost") => {
   // create a server with express
   const app = express()
 

test/test.js

@@ -2,6 +2,8 @@ const assert = require("assert")
 const fs = require("fs")
 const http = require("http")
 const https = require("https")
+const tls = require("tls")
+const net = require("net")
 
 const sinon = require("sinon")
 
@@ -148,6 +150,33 @@ describe("Testing certs", function() {
       done()
     })()
   })
+
+  it("works with environment domain", function(done) {
+    (async() => {
+      // set the environment domain
+      process.env.DOMAIN = "192.168.0.1"
+
+      // Get the cert
+      const appCerts = await app.getCerts()
+
+      // Configure the cert to be able to read it
+      const secureContext = tls.createSecureContext({
+        cert: appCerts.cert
+      })
+      const secureSocket = new tls.TLSSocket(new net.Socket(), {
+        secureContext
+      })
+
+      // Read the cert and parse out the domain
+      const cert = secureSocket.getCertificate()
+      const certDomain = cert.subjectaltname.split(":")[1]
+
+      // Compare the domains
+      assert(certDomain === process.env.DOMAIN)
+
+      done()
+    })()
+  })
 })
 
 // TESTS MODULE