fix(stateConfig): use localhost rather than ip for dev

__tests__/server/utils/__snapshots__/stateConfig.spec.js.snap

@@ -2,13 +2,13 @@
 
 exports[`stateConfig methods stateConfig with dev endpoint config should show dev endpoint supplied config 1`] = `
 Object {
-  "someOtherApiUrl": "http://127.0.0.1:3002/some-other-api",
+  "someOtherApiUrl": "http://localhost:3002/some-other-api",
 }
 `;
 
 exports[`stateConfig methods stateConfig with dev endpoint config should show dev endpoint supplied config 2`] = `
 Object {
-  "someOtherApiUrl": "http://127.0.0.1:3002/some-other-api",
+  "someOtherApiUrl": "http://localhost:3002/some-other-api",
 }
 `;
 
@@ -156,15 +156,15 @@ exports[`stateConfig methods stateConfig with priorities of should prioritize de
 Object {
   "anotherSetting": "another-fake-setting",
   "fakeSetting": "client-fake-setting",
-  "someOtherApiUrl": "http://127.0.0.1:3002/some-other-api",
+  "someOtherApiUrl": "http://localhost:3002/some-other-api",
 }
 `;
 
 exports[`stateConfig methods stateConfig with priorities of should prioritize dev endpoint over env 2`] = `
 Object {
   "anotherSetting": "another-fake-setting",
   "fakeSetting": "server-fake-setting",
-  "someOtherApiUrl": "http://127.0.0.1:3002/some-other-api",
+  "someOtherApiUrl": "http://localhost:3002/some-other-api",
 }
 `;
 

__tests__/server/utils/stateConfig.spec.js

@@ -15,7 +15,6 @@
  */
 
 jest.mock('fs', () => ({ existsSync: jest.fn() }));
-jest.mock('ip', () => ({ address: jest.fn() }));
 jest.mock('fake/path/.dev/endpoints/index.js', () => jest.fn(), { virtual: true });
 jest.mock('../../../src/server/utils/envVarAllowList', () => [
   'ONE_CLIENT_FAKE_SETTING',
@@ -33,15 +32,12 @@ describe('stateConfig methods', () => {
   let restoreModuleStateConfig;
   let backupModuleStateConfig;
   let fs;
-  let ip;
 
   const originalEnvVars = process.env;
 
   const reloadMocks = () => {
     fs = require('fs');
-    ip = require('ip');
     jest.spyOn(process, 'cwd').mockImplementation(() => 'fake/path/');
-    ip.address.mockImplementation(() => '127.0.0.1');
     fs.existsSync.mockImplementation(() => false);
     process.env.ONE_CONFIG_ENV = 'qa';
     provideStateConfig = {
@@ -209,7 +205,7 @@ describe('stateConfig methods', () => {
           getClientStateConfig,
           getServerStateConfig,
         } = require('../../../src/server/utils/stateConfig'));
-        expect(getClientStateConfig().leadingSlashApiUrl).toEqual('http://127.0.0.1:3002/leading-slash-api');
+        expect(getClientStateConfig().leadingSlashApiUrl).toEqual('http://localhost:3002/leading-slash-api');
       });
     });
     describe('with env vars', () => {

src/server/utils/stateConfig.js

@@ -14,7 +14,6 @@
  * permissions and limitations under the License.
  */
 
-import ip from 'ip';
 import fs from 'fs';
 import path from 'path';
 import url from 'url';
@@ -72,14 +71,13 @@ const pathToDevEndpoints = path.join(process.cwd(), '.dev', 'endpoints', 'index.
 const stateConfigFromDevEndpoints = {};
 
 if (process.env.NODE_ENV === 'development' && fs.existsSync(pathToDevEndpoints)) {
-  const ipAddress = ip.address();
   const { SERVICES_PORT = 3002 } = process.env;
   // eslint-disable-next-line global-require,import/no-dynamic-require
   const devEndpoints = require(pathToDevEndpoints)();
   Object.entries(devEndpoints).forEach(([configName, { devProxyPath }]) => {
     const value = url.format({
       protocol: 'http',
-      hostname: ipAddress,
+      hostname: 'localhost',
       port: SERVICES_PORT,
       pathname: devProxyPath,
     });