Merge pull request TencentBlueKing#11235 from yongyiduan/issue_11234

[bugfix] okhttp3 Response 不主动关闭会有潜在的内存泄露问题 TencentBlueKing#11234
yjieliang · Nov 20, 2024 · c4af3cb · c4af3cb
2 parents 9182226 + 76c2180
commit c4af3cb
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/.../biz-process/src/main/kotlin/com/tencent/devops/process/engine/control/CallBackControl.kt b/.../biz-process/src/main/kotlin/com/tencent/devops/process/engine/control/CallBackControl.kt
@@ -31,6 +31,7 @@ import com.fasterxml.jackson.databind.ObjectMapper
 import com.tencent.devops.common.api.exception.RemoteServiceException
 import com.tencent.devops.common.api.util.Watcher
 import com.tencent.devops.common.api.util.timestampmilli
+import com.tencent.devops.common.archive.util.closeQuietly
 import com.tencent.devops.common.client.Client
 import com.tencent.devops.common.event.pojo.pipeline.PipelineBuildStatusBroadCastEvent
 import com.tencent.devops.common.pipeline.Model
@@ -319,7 +320,7 @@ class CallBackControl @Autowired constructor(
         try {
             breaker.executeCallable {
                 HttpRetryUtils.retry(MAX_RETRY_COUNT) {
-                    callbackClient.newCall(request).execute()
+                    callbackClient.newCall(request).execute().closeQuietly()
                 }
                 if (callBack.failureTime != null) {
                     projectPipelineCallBackService.updateFailureTime(