ensuring to close client after request

crashappsec · Feb 15, 2024 · eed75bb · eed75bb
1 parent e0ed603
commit eed75bb
Showing 1 changed file with 10 additions and 7 deletions.
diff --git a/nimutils/net.nim b/nimutils/net.nim
@@ -222,10 +222,13 @@ proc safeRequest*(url: Uri | string,
                                 timeout       = timeout,
                                 pinnedCert    = pinnedCert,
                                 disallowHttp  = disallowHttp)
-  return client.safeRequest(url               = uri,
-                            httpMethod        = httpMethod,
-                            body              = body,
-                            headers           = headers,
-                            multipart         = multipart,
-                            retries           = retries,
-                            firstRetryDelayMs = firstRetryDelayMs)
+  try:
+    return client.safeRequest(url               = uri,
+                              httpMethod        = httpMethod,
+                              body              = body,
+                              headers           = headers,
+                              multipart         = multipart,
+                              retries           = retries,
+                              firstRetryDelayMs = firstRetryDelayMs)
+  finally:
+    client.close()