Merge pull request #515 from codacy/improve-timeout-http-helper

increase connection and read timeouts
codacy · Nov 25, 2024 · 880c138 · 880c138
2 parents b499a22 + aeb089e
commit 880c138
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/core/src/main/scala/com/codacy/analysis/core/utils/HttpHelper.scala b/core/src/main/scala/com/codacy/analysis/core/utils/HttpHelper.scala
@@ -7,8 +7,8 @@ import scalaj.http.{Http, HttpOptions, HttpRequest, HttpResponse}
 
 class HttpHelper(apiUrl: String, extraHeaders: Map[String, String], allowUnsafeSSL: Boolean) {
 
-  private lazy val connectionTimeoutMs = 2000
-  private lazy val readTimeoutMs = 5000
+  private lazy val connectionTimeoutMs = 5000
+  private lazy val readTimeoutMs = 15000
 
   private val remoteUrl = apiUrl + "/2.0"
 

diff --git a/core/src/test/scala/com.codacy.analysis.core/upload/ResultsUploaderSpec.scala b/core/src/test/scala/com.codacy.analysis.core/upload/ResultsUploaderSpec.scala
@@ -155,7 +155,7 @@ class ResultsUploaderSpec extends Specification with NoLanguageFeatures with Moc
       val testDuplication =
         Seq(DuplicationResult(language, DuplicationAnalysis.Success(Set.empty, Set(testClone(1), testClone(2)))))
 
-      uploader.sendResults(Seq.empty, Seq.empty, testDuplication) must beRight.awaitFor(10.seconds)
+      uploader.sendResults(Seq.empty, Seq.empty, testDuplication) must beRight.awaitFor(30.seconds)
 
       there were no(codacyClient).sendRemoteIssues(
         ArgumentMatchers.any[String],