From d8a1c495e27bb2561fda29bc2a13764e2bb16595 Mon Sep 17 00:00:00 2001
From: Joao Machado <machado@codacy.com>
Date: Mon, 16 May 2022 12:57:17 +0100
Subject: [PATCH] fix: Use the http-timeout for the read-timeout only CY-6106

---
 README.md                                                   | 2 +-
 .../codacy/configuration/parser/ConfigurationParser.scala   | 4 +++-
 src/main/scala/com/codacy/rules/ConfigurationRules.scala    | 2 +-
 src/test/scala/com/codacy/rules/ReportRulesSpec.scala       | 6 +++---
 4 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/README.md b/README.md
index d4da96de..46e27e4c 100644
--- a/README.md
+++ b/README.md
@@ -35,7 +35,7 @@ Usage: codacy-coverage-reporter report
   --project-name | -p  <your project name>
   --codacy-api-base-url  <the base URL for the Codacy API>
   --commit-uuid  <your commitUUID>
-  --http-timeout  <Sets a specified timeout value, in milliseconds, to be used when interacting with Codacy API>
+  --http-timeout  <Sets a specified read timeout value, in milliseconds, to be used when interacting with Codacy API>
   --skip | -s  <skip if token isn't defined>
   --language | -l  <language associated with your coverage report>
   --coverage-reports | -r  <your project coverage file name>
diff --git a/src/main/scala/com/codacy/configuration/parser/ConfigurationParser.scala b/src/main/scala/com/codacy/configuration/parser/ConfigurationParser.scala
index 6bf1c8c0..54131dea 100644
--- a/src/main/scala/com/codacy/configuration/parser/ConfigurationParser.scala
+++ b/src/main/scala/com/codacy/configuration/parser/ConfigurationParser.scala
@@ -87,7 +87,9 @@ case class BaseCommandConfig(
     codacyApiBaseUrl: Option[String],
     @ValueDescription("your commitUUID")
     commitUUID: Option[String],
-    @ValueDescription("Sets a specified timeout value, in milliseconds, to be used when interacting with Codacy API")
+    @ValueDescription(
+      "Sets a specified read timeout value, in milliseconds, to be used when interacting with Codacy API"
+    )
     httpTimeout: Int = 10000,
     @Name("s") @ValueDescription("skip if token isn't defined")
     skip: Int @@ Counter = Tag.of(0),
diff --git a/src/main/scala/com/codacy/rules/ConfigurationRules.scala b/src/main/scala/com/codacy/rules/ConfigurationRules.scala
index fd887457..334e8344 100644
--- a/src/main/scala/com/codacy/rules/ConfigurationRules.scala
+++ b/src/main/scala/com/codacy/rules/ConfigurationRules.scala
@@ -72,7 +72,7 @@ class ConfigurationRules(cmdConfig: CommandConfiguration, envVars: Map[String, S
         baseConfig.codacyApiBaseUrl.getOrElse(getApiBaseUrl),
         commitUUID,
         baseConfig.debugValue,
-        timeout = RequestTimeout(baseConfig.httpTimeout, baseConfig.httpTimeout)
+        timeout = RequestTimeout(connTimeoutMs = 1000, readTimeoutMs = baseConfig.httpTimeout)
       )
       validatedConfig <- validateBaseConfigUrl(baseConf)
     } yield {
diff --git a/src/test/scala/com/codacy/rules/ReportRulesSpec.scala b/src/test/scala/com/codacy/rules/ReportRulesSpec.scala
index 1551f0bc..8e87abaf 100644
--- a/src/test/scala/com/codacy/rules/ReportRulesSpec.scala
+++ b/src/test/scala/com/codacy/rules/ReportRulesSpec.scala
@@ -37,7 +37,7 @@ class ReportRulesSpec extends WordSpec with Matchers with PrivateMethodTester wi
         apiBaseUrl,
         None,
         debug = false,
-        timeout = RequestTimeout(10000, 10000)
+        timeout = RequestTimeout(1000, 10000)
       )
 
     def assertCodacyCoverage(
@@ -88,7 +88,7 @@ class ReportRulesSpec extends WordSpec with Matchers with PrivateMethodTester wi
           any[String],
           any[CoverageReport],
           anyBoolean,
-          Some(RequestTimeout(10000, 10000))
+          Some(RequestTimeout(1000, 10000))
         ) returns FailedResponse("Failed to send report")
 
         assertCodacyCoverage(coverageServices, List("src/test/resources/dotcover-example.xml"), success = false)
@@ -103,7 +103,7 @@ class ReportRulesSpec extends WordSpec with Matchers with PrivateMethodTester wi
         any[String],
         any[CoverageReport],
         anyBoolean,
-        Some(RequestTimeout(10000, 10000))
+        Some(RequestTimeout(1000, 10000))
       ) returns SuccessfulResponse(RequestSuccess("Success"))
 
       assertCodacyCoverage(coverageServices, List("src/test/resources/dotcover-example.xml"), success = true)