fix(react): timeout after 20 min waiting for remote

[koenvangeert]

Current Behavior

When using module federation the server will close after 20 minutes with the following error:

.../node_modules/@nx/react/src/executors/module-federation-dev-server/module-federation-dev-server.impl.js:98
                throw new Error(`Timed out waiting for remote to start. Check above for any errors.`);
                      ^

Error: Timed out waiting for remote to start. Check above for any errors.
    at .../node_modules/@nx/react/src/executors/module-federation-dev-server/module-federation-dev-server.impl.js:98:23
    at Generator.throw (<anonymous>)
    at rejected (.../node_modules/tslib/tslib.js:165:69)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

Expected Behavior

The server keeps on running

Cause

The following pr caused the bug to occur: #17192
The webpack dev server doesn't bind to 127.0.0.1 by default.
It only binds to localhost and ::1.

Related Issue(s)

Fixes #17354

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
nx-dev	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 25, 2023 1:56pm

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit ee7da89. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch

---

✅ Successfully ran 8 targets

• linux nx affected --target=e2e --base=e6c67e41ec373781fc05363925f1217f5840de60 --head=ee7da8907ede170de678ba2307b6a992fb73a7cb --parallel=1
• linux nx affected --target=test --base=e6c67e41ec373781fc05363925f1217f5840de60 --head=ee7da8907ede170de678ba2307b6a992fb73a7cb --parallel=1
• linux nx affected --target=build --base=e6c67e41ec373781fc05363925f1217f5840de60 --head=ee7da8907ede170de678ba2307b6a992fb73a7cb --parallel=3
• linux nx run-many -t check-imports check-commit check-lock-files check-codeowners documentation --parallel=1 --no-dte
• linux nx affected --target=lint --base=e6c67e41ec373781fc05363925f1217f5840de60 --head=ee7da8907ede170de678ba2307b6a992fb73a7cb --parallel=3
• nx-cloud record -- nx format:check --base=e6c67e41ec373781fc05363925f1217f5840de60 --head=ee7da8907ede170de678ba2307b6a992fb73a7cb
• linux nx documentation --no-dte
• macos nx affected -t e2e-macos --parallel=1 --base=e6c67e41ec373781fc05363925f1217f5840de60 --head=ee7da8907ede170de678ba2307b6a992fb73a7cb

---

Sent with 💌 from NxCloud.

[xiongemi]

thanks for submitting this pr. i think host: 'localhost' was already removed. i am going to close this pr.

[vadimka123]

@xiongemi , removing host: 'localhost' cause issue
Returning back will fix this bug
I manually fixing every time now by updating relevant file in my local node_modules with this change
But I want to have merged this change ASAP

[Coly010]

Let me take a look at this :)

[DheerajMahra]

Hi team, please mentioned the release into which this issue is fixed

[DheerajMahra]

Ok, the fix is listed in 16.6.0 pre-release - https://github.com/nrwl/nx/releases/tag/16.6.0-beta.6

When are you planning to release the stable 16.6.0?

[github-actions]

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.