refactor: remove is-absolute-url in favor of method (#3609)

Author: snitin315

lib/Server.js

@@ -49,6 +49,18 @@ class Server {
     };
   }
 
+  // eslint-disable-next-line class-methods-use-this
+  static isAbsoluteURL(URL) {
+    // Don't match Windows paths `c:\`
+    if (/^[a-zA-Z]:\\/.test(URL)) {
+      return false;
+    }
+
+    // Scheme: https://tools.ietf.org/html/rfc3986#section-3.1
+    // Absolute URL: https://tools.ietf.org/html/rfc3986#section-4.3
+    return /^[a-zA-Z][a-zA-Z\d+\-.]*:/.test(URL);
+  }
+
   static async getHostname(hostname) {
     if (hostname === "local-ip") {
       return (await internalIp.v4()) || (await internalIp.v6()) || "0.0.0.0";
@@ -523,10 +535,8 @@ class Server {
     }
 
     if (options.static) {
-      const isAbsoluteUrl = require("is-absolute-url");
-
       options.static.forEach((staticOption) => {
-        if (isAbsoluteUrl(staticOption.directory)) {
+        if (Server.isAbsoluteURL(staticOption.directory)) {
           throw new Error("Using a URL as static.directory is not supported");
         }
 
@@ -1193,7 +1203,6 @@ class Server {
   }
 
   openBrowser(defaultOpenTarget) {
-    const isAbsoluteUrl = require("is-absolute-url");
     const open = require("open");
 
     Promise.all(
@@ -1203,7 +1212,7 @@ class Server {
         if (item.target === "<url>") {
           openTarget = defaultOpenTarget;
         } else {
-          openTarget = isAbsoluteUrl(item.target)
+          openTarget = Server.isAbsoluteURL(item.target)
             ? item.target
             : new URL(item.target, defaultOpenTarget).toString();
         }

package-lock.json

@@ -45,7 +45,6 @@
     "http-proxy-middleware": "^2.0.0",
     "internal-ip": "^6.2.0",
     "ipaddr.js": "^2.0.1",
-    "is-absolute-url": "^3.0.3",
     "killable": "^1.0.1",
     "open": "^8.0.9",
     "p-retry": "^4.5.0",