fix: Increase api timeout limit to 120s

Signed-off-by: Lei Jin <lei.jin@lacework.net>
lacework · Sep 5, 2024 · 4067cb5 · 4067cb5
1 parent 5eed888
commit 4067cb5
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 8 deletions.
diff --git a/api/client.go b/api/client.go
@@ -36,8 +36,8 @@ import (
 )
 
 const (
-	defaultTimeout    = 60 * time.Second
-	defaultTLSTimeout = 63 * time.Second
+	defaultTimeout    = 120 * time.Second
+	defaultTLSTimeout = 123 * time.Second
 )
 
 type Client struct {

diff --git a/integration/container_vulnerability_test.go b/integration/container_vulnerability_test.go
@@ -220,9 +220,7 @@ func _TestContainerVulnerabilityCommandScanFailOnSeverity(t *testing.T) {
 		"STDERR doesn't match")
 }
 
-// Disabled due to the api timeout issue
-// TODO: re-enable when timeout is fixed
-func _TestContainerVulnerabilityCommandScanFailOnFixable(t *testing.T) {
+func TestContainerVulnerabilityCommandScanFailOnFixable(t *testing.T) {
 	home := createTOMLConfigFromCIvars()
 	defer os.RemoveAll(home)
 	_, err, exitcode := LaceworkCLIWithHome(home,
@@ -235,9 +233,7 @@ func _TestContainerVulnerabilityCommandScanFailOnFixable(t *testing.T) {
 		"STDERR doesn't match")
 }
 
-// Disabled due to the api timeout issue
-// TODO: re-enable when timeout is fixed
-func _TestContainerVulnerabilityCommandScanHumanReadablePollGenerateHtml(t *testing.T) {
+func TestContainerVulnerabilityCommandScanHumanReadablePollGenerateHtml(t *testing.T) {
 	// create a temporal directory to check that the HTML file is deployed
 	home := createTOMLConfigFromCIvars()
 	defer os.RemoveAll(home)