Merge pull request #1 from blockchain-etl/bug/memory_leak_in_rpc

Fix memory leak in request.py
blockchain-etl · Aug 27, 2020 · 166ce8c · 166ce8c
2 parents c7e1575 + e2b79cf
commit 166ce8c
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/tezosetl/rpc/request.py b/tezosetl/rpc/request.py
@@ -21,10 +21,10 @@ def make_post_request(endpoint_uri, data, *args, **kwargs):
     return response.content
 
 
-def make_get_request(endpoint_uri, *args, **kwargs):
+def make_get_request(url, path, *args, **kwargs):
     kwargs.setdefault('timeout', 10)
-    session = _get_session(endpoint_uri)
-    response = session.get(endpoint_uri, *args, **kwargs)
+    session = _get_session(url)
+    response = session.get(url + path, *args, **kwargs)
     response.raise_for_status()
 
     return response.content
diff --git a/tezosetl/rpc/tezos_rpc.py b/tezosetl/rpc/tezos_rpc.py
@@ -34,7 +34,8 @@ def __init__(self, provider_uri, timeout=60):
 
     def get(self, endpoint):
         raw_response = make_get_request(
-            self.provider_uri + endpoint,
+            self.provider_uri,
+            endpoint,
             timeout=self.timeout
         )