feat: remove internalIP methods and update references to findIp

Author: bjohansebas

lib/Server.js

@@ -475,24 +475,6 @@ class Server {
     }
   }
 
-  // TODO remove me in the next major release, we have `findIp`
-  /**
-   * @param {"v4" | "v6"} family family
-   * @returns {Promise<string | undefined>} internal API
-   */
-  static async internalIP(family) {
-    return Server.findIp(family, false);
-  }
-
-  // TODO remove me in the next major release, we have `findIp`
-  /**
-   * @param {"v4" | "v6"} family family
-   * @returns {string | undefined} internal IP
-   */
-  static internalIPSync(family) {
-    return Server.findIp(family, false);
-  }
-
   /**
    * @param {Host} hostname hostname
    * @returns {Promise<string>} resolved hostname

test/e2e/allowed-hosts.test.js

@@ -1657,7 +1657,7 @@ describe("allowed hosts", () => {
     });
 
     it("should always allow value from the `host` options if options.allowedHosts is auto", async () => {
-      const networkIP = Server.internalIPSync("v4");
+      const networkIP = Server.findIp("v4", false);
       const options = {
         host: networkIP,
         allowedHosts: "auto",

test/e2e/web-socket-server-url.test.js

@@ -113,7 +113,7 @@ describe("web socket server URL", () => {
     it(`should work behind proxy, when hostnames are different and ports are same ("${webSocketServer}")`, async () => {
       const devServerHost = "127.0.0.1";
       const devServerPort = port1;
-      const proxyHost = Server.internalIPSync("v4");
+      const proxyHost = Server.findIp("v4", false);
       const proxyPort = port1;
 
       const compiler = webpack(config);
@@ -209,7 +209,7 @@ describe("web socket server URL", () => {
     it(`should work behind proxy, when hostnames are different and ports are different ("${webSocketServer}")`, async () => {
       const devServerHost = "localhost";
       const devServerPort = port1;
-      const proxyHost = Server.internalIPSync("v4");
+      const proxyHost = Server.findIp("v4", false);
       const proxyPort = port2;
 
       const compiler = webpack(config);
@@ -310,7 +310,7 @@ describe("web socket server URL", () => {
     it(`should work behind proxy, when the "host" option is "local-ip" and the "port" option is "auto" ("${webSocketServer}")`, async () => {
       process.env.WEBPACK_DEV_SERVER_BASE_PORT = 40000;
 
-      const proxyHost = Server.internalIPSync("v4");
+      const proxyHost = Server.findIp("v4", false);
       const proxyPort = port2;
 
       const compiler = webpack(config);
@@ -2006,7 +2006,7 @@ describe("web socket server URL", () => {
     });
 
     it(`should work when "host" option is IPv4 ("${webSocketServer}")`, async () => {
-      const hostname = Server.internalIPSync("v4");
+      const hostname = Server.findIp("v4", false);
       const compiler = webpack(config);
       const devServerOptions = {
         webSocketServer,
@@ -2074,7 +2074,7 @@ describe("web socket server URL", () => {
     });
 
     it(`should work when "host" option is "local-ip" ("${webSocketServer}")`, async () => {
-      const hostname = Server.internalIPSync("v4");
+      const hostname = Server.findIp("v4", false);
       const compiler = webpack(config);
       const devServerOptions = {
         webSocketServer,
@@ -2143,7 +2143,7 @@ describe("web socket server URL", () => {
     });
 
     it(`should work when "host" option is "local-ipv4" ("${webSocketServer}")`, async () => {
-      const hostname = Server.internalIPSync("v4");
+      const hostname = Server.findIp("v4", false);
       const compiler = webpack(config);
       const devServerOptions = {
         webSocketServer,

test/server/open-option.test.js

@@ -5,7 +5,7 @@ const Server = require("../../lib/Server");
 const config = require("../fixtures/simple-config/webpack.config");
 const port = require("../ports-map")["open-option"];
 
-const internalIPv4 = Server.internalIPSync("v4");
+const internalIPv4 = Server.findIp("v4", false);
 
 let open;
 

types/lib/Server.d.ts

@@ -1141,16 +1141,6 @@ declare class Server<
     gatewayOrFamily: string,
     isInternal?: boolean | undefined,
   ): string | undefined;
-  /**
-   * @param {"v4" | "v6"} family family
-   * @returns {Promise<string | undefined>} internal API
-   */
-  static internalIP(family: "v4" | "v6"): Promise<string | undefined>;
-  /**
-   * @param {"v4" | "v6"} family family
-   * @returns {string | undefined} internal IP
-   */
-  static internalIPSync(family: "v4" | "v6"): string | undefined;
   /**
    * @param {Host} hostname hostname
    * @returns {Promise<string>} resolved hostname