fix(deps): update dependency undici to v8#728
Conversation
commit: |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
ddfc9af to
34e1186
Compare
|
@copilot fix this ci fails |
In undici v8, ProxyAgent defaults proxyTunnel to true, which means CONNECT tunneling is used even for HTTP-to-HTTP proxy requests. These tunnel connections keep the proxy server alive, preventing proxyServer.close() from completing within the 10s afterAll timeout. Adding proxyServer.closeAllConnections() before proxyServer.close() forcefully terminates the tunnel connections and allows cleanup. Agent-Logs-Url: https://github.com/node-modules/urllib/sessions/4c3edcc4-7c2f-4451-b8f8-b324a333958d Co-authored-by: fengmk2 <156269+fengmk2@users.noreply.github.com>
Fixed in da2f3be. The CI failure was caused by undici v8 defaulting Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
const proxyServer = setup();
|
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. |
|
@copilot still fails |
…ll timeout In undici v8, ProxyAgent and Agent maintain persistent connections (especially H2 connections) that keep the proxy server's CONNECT tunnels alive. Close all dispatcher instances at the end of each test to allow clean proxy server shutdown within the afterAll timeout. Agent-Logs-Url: https://github.com/node-modules/urllib/sessions/bcba591d-864f-4402-8861-c5be26f032ba Co-authored-by: fengmk2 <156269+fengmk2@users.noreply.github.com>
Fixed in a54db5c. The previous fix only added Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
// Create proxy server // Create HTTP ta` (dns block)
|
Renovate Ignore NotificationBecause you closed this PR without merging, Renovate will ignore this update. You will not get PRs for any future If you accidentally closed this PR, or if you changed your mind: rename this PR to get a fresh replacement PR. |
test/options.dispatcher.test.tsafterAll hook timeoutproxyServer.closeAllConnections()in afterAll