Filip/playwright
1
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases Wiki Activity

also add socks proxy

Browse source
This commit is contained in:
Simon Knott 2024-09-27 14:37:24 +02:00
parent 988da93cc8
commit 6492999506
No known key found for this signature in database
GPG key ID: 8CEDC00028084AEC
2 changed files with 5 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
packages/playwright-core/src/server/fetch.ts
View file

@ -508,6 +508,10 @@ export abstract class APIRequestContext extends SdkObject {
}
});
// when using socks proxy, having the socket means the connection got established
if (agent instanceof SocksProxyAgent)
tcpConnectionAt ??= monotonicTime();
serverIPAddress = socket.remoteAddress;
serverPort = socket.remotePort;
});

3
tests/library/har.spec.ts
View file

@ -868,8 +868,7 @@ it('should include timings when using http proxy', async ({ contextFactory, serv
expect(log.entries[0].timings.connect).toBeGreaterThan(0);
});
// socks proxy library doesnt emit an event to know when the TCP connection starts
it.fail('should include timings when using socks proxy', async ({ contextFactory, server, socksPort }, testInfo) => {
it('should include timings when using socks proxy', async ({ contextFactory, server, socksPort }, testInfo) => {
const { page, getLog } = await pageWithHar(contextFactory, testInfo, { proxy: { server: `socks5://localhost:${socksPort}` } });
const response = await page.request.get(server.EMPTY_PAGE);
await response.body();