deprecated memoize and added lru_cache

Author: linglp

synapseclient/client.py

@@ -118,7 +118,6 @@
     id_of,
     get_properties,
     MB,
-    memoize,
     is_json,
     extract_synapse_id_from_query,
     find_data_file_handle,
@@ -525,7 +524,7 @@ def login(
             cached_sessions.set_most_recent_user(self.credentials.username)
 
         if not silent:
-            profile = self.getUserProfile(refresh=True)
+            profile = self.getUserProfile()
             # TODO-PY3: in Python2, do we need to ensure that this is encoded in utf-8
             self.logger.info(
                 "Welcome, %s!\n"
@@ -639,21 +638,19 @@ def invalidateAPIKey(self):
         if self._is_logged_in():
             self.restDELETE("/secretKey", endpoint=self.authEndpoint)
 
-    @memoize
-    def getUserProfile(self, id=None, sessionToken=None, refresh=False):
+    @functools.lru_cache()
+    def getUserProfile(self, id=None, sessionToken=None):
         """
         Get the details about a Synapse user.
         Retrieves information on the current user if 'id' is omitted.
         :param id:           The 'userId' (aka 'ownerId') of a user or the userName
         :param sessionToken: The session token to use to find the user profile
-        :param refresh:      If set to True will always fetch the data from Synape otherwise will use cached information
         :returns: The user profile for the user of interest.
 
         Example::
             my_profile = syn.getUserProfile()
             freds_profile = syn.getUserProfile('fredcommo')
         """
-
         try:
             # if id is unset or a userID, this will succeed
             id = "" if id is None else int(id)
@@ -674,7 +671,6 @@ def getUserProfile(self, id=None, sessionToken=None, refresh=False):
                     else:  # no break
                         raise ValueError('Can\'t find user "%s": ' % id)
         uri = "/userProfile/%s" % id
-
         return UserProfile(
             **self.restGET(
                 uri, headers={"sessionToken": sessionToken} if sessionToken else None
@@ -2589,7 +2585,7 @@ def _download_from_URL(
                     else:
                         mode = "wb"
                         previouslyTransferred = 0
-                        sig = hashlib.md5()
+                        sig = hashlib.md5(usedforsecurity=False)
 
                     try:
                         with open(temp_destination, mode) as fd:
@@ -4681,7 +4677,6 @@ def _generate_headers(self, headers=None):
 
         if headers is None:
             headers = dict(self.default_headers)
-
         headers.update(synapseclient.USER_AGENT)
 
         return headers
@@ -4717,6 +4712,7 @@ def _rest_call(
         uri, headers = self._build_uri_and_headers(
             uri, endpoint=endpoint, headers=headers
         )
+
         retryPolicy = self._build_retry_policy(retryPolicy)
         requests_session = requests_session or self._requests_session
 
@@ -4733,6 +4729,7 @@ def _rest_call(
             verbose=self.debug,
             **retryPolicy,
         )
+
         self._handle_synapse_http_error(response)
         return response
 

synapseclient/core/utils.py

@@ -11,7 +11,6 @@
 import collections.abc
 import datetime
 import errno
-import functools
 import hashlib
 import importlib
 import inspect
@@ -51,7 +50,7 @@ def md5_for_file(filename, block_size=2 * MB, callback=None):
     :returns: The MD5
     """
 
-    md5 = hashlib.md5()
+    md5 = hashlib.md5(usedforsecurity=False)
     with open(filename, "rb") as f:
         while True:
             if callback:
@@ -644,21 +643,6 @@ def extract_synapse_id_from_query(query):
         raise ValueError('Couldn\'t extract synapse ID from query: "%s"' % query)
 
 
-# Derived from https://wiki.python.org/moin/PythonDecoratorLibrary#Memoize
-def memoize(obj):
-    cache = obj._memoize_cache = {}
-
-    @functools.wraps(obj)
-    def memoizer(*args, **kwargs):
-        refresh = kwargs.pop("refresh", False)
-        key = str(args) + str(kwargs)
-        if refresh or key not in cache:
-            cache[key] = obj(*args, **kwargs)
-        return cache[key]
-
-    return memoizer
-
-
 def printTransferProgress(
     transferred,
     toBeTransferred,