fix(proxy): ensure the host field is correct when using proxy#40797
fix(proxy): ensure the host field is correct when using proxy#40797Archkon wants to merge 3 commits into
Conversation
|
Please start with opening an issue that describes the problem, so that we can evaluate it. |
yury-s
left a comment
There was a problem hiding this comment.
Looks good, but please add a test for this. See e.g. 'should use env proxy with connectOverCDP discovery request' in tests/library/chromium/connect-over-cdp.spec.ts.
@microsoft-github-policy-service agree |
Test results for "tests 1"3 flaky41774 passed, 850 skipped Merge workflow run. |
Test results for "MCP"3 failed 7083 passed, 1104 skipped Merge workflow run. |
The host field should be set according target url not the proxy url
refer to https://www.rfc-editor.org/rfc/rfc9112#section-3.2.2