Skip to content

Commit 564cfa2

Browse files
authored
Remove ResponseHeaderTimeout from default http transport (#1827)
Signed-off-by: Daniel Valdivia <18384552+dvaldivia@users.noreply.github.com>
1 parent 398ab02 commit 564cfa2

File tree

2 files changed

+9
-6
lines changed

2 files changed

+9
-6
lines changed

portal-ui/tests/permissions-2/diagnostics.ts

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,9 @@ test("Start New Diagnostic button exists after Diagnostic is completed", async (
8080
.useRole(roles.diagnostics)
8181
.navigateTo("http://localhost:9090/support/diagnostics")
8282
.click(elements.startDiagnosticButton)
83-
.wait(2000)
83+
.wait(3000)
84+
.expect(elements.downloadButton.exists)
85+
.ok()
8486
.expect(elements.startNewDiagnosticButton.exists)
8587
.ok();
8688
});
@@ -90,6 +92,8 @@ test("Start New Diagnostic button is clickable after Diagnostic is completed", a
9092
.useRole(roles.diagnostics)
9193
.navigateTo("http://localhost:9090/support/diagnostics")
9294
.click(elements.startDiagnosticButton)
93-
.wait(2000)
95+
.wait(3000)
96+
.expect(elements.downloadButton.exists)
97+
.ok()
9498
.click(elements.startNewDiagnosticButton);
9599
});

restapi/tls.go

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -32,15 +32,14 @@ func prepareSTSClientTransport(insecure bool) *http.Transport {
3232
DefaultTransport := &http.Transport{
3333
Proxy: http.ProxyFromEnvironment,
3434
DialContext: (&net.Dialer{
35-
Timeout: 5 * time.Second,
35+
Timeout: 10 * time.Second,
3636
KeepAlive: 15 * time.Second,
3737
}).DialContext,
3838
MaxIdleConns: 1024,
3939
MaxIdleConnsPerHost: 1024,
40-
ResponseHeaderTimeout: 60 * time.Second,
41-
IdleConnTimeout: 60 * time.Second,
40+
IdleConnTimeout: 90 * time.Second,
4241
TLSHandshakeTimeout: 10 * time.Second,
43-
ExpectContinueTimeout: 1 * time.Second,
42+
ExpectContinueTimeout: 10 * time.Second,
4443
DisableCompression: true,
4544
TLSClientConfig: &tls.Config{
4645
// Can't use SSLv3 because of POODLE and BEAST

0 commit comments

Comments
 (0)