feat(api): add host option in launchServer options

[cavivie]

Closes 30998.

[cavivie]

@microsoft-github-policy-service agree

[mxschmitt]

LGTM % comments

[cavivie]

@mxschmitt Is this dirty error caused by me pushing twice quickly?

[mxschmitt]

@mxschmitt Is this dirty error caused by me pushing twice quickly?

You need to run npm run build, it will then update some generated files.

[cavivie]

BTW, I found that although the ws server pointed out that the ws url was localhost, it actually listened at 0.0.0.0. This is because the http server listen uses net.Server listen. When the host is omitted (undefined) in this method, the unspecified address will be used.

https://github.com/microsoft/playwright/blob/main/packages/playwright-core/src/utils/wsServer.ts#L57-L83

https://nodejs.org/api/net.html#serverlistenport-host-backlog-callback

To fix it, we need to do either of these things:

1. We may need to open a new bug issue for this and fix it: set hostname to localhost instead of undefined, line to change, because undefined means that ws net servers listen on the unspecified address ipv4(0.0.0.0)/ipv6(::).
2. We only change the document description to unspecified, and chang the hostname in the ws url to undefined value to 0.0.0.0 but notlocalhost, line to change.

[cavivie]

2. We only change the document description to unspecified, and chang the hostname in the ws url to undefined value to 0.0.0.0 but notlocalhost, line to change.

Finally, I followed this to keep compatibility, what do you think?

[github-actions]

Test results for "tests 1"

2 flaky

⚠️ [chromium-library] › library/trace-viewer.spec.ts:240:1 › should have network requests
⚠️ [webkit-page] › page/workers.spec.ts:243:3 › should support offline

27061 passed, 609 skipped
✔️✔️✔️

Merge workflow run.