From e9dde2925b9f52b0ef94b2bd76cdd35512ae89d0 Mon Sep 17 00:00:00 2001 From: Ade Lee Date: Wed, 17 Feb 2021 16:37:08 -0500 Subject: [PATCH] Add TODO to remove md5 wrapper Added TODO to make sure we remember to remove the md5 wrapper when the minimum python distribution supports it. Change-Id: I2a489b0f645c77d58c7de070f88c1d79832e92cf --- openstack/utils.py | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/openstack/utils.py b/openstack/utils.py index 8adbc54b6..5fff2b3e5 100644 --- a/openstack/utils.py +++ b/openstack/utils.py @@ -239,6 +239,10 @@ try: # Python distributions that support a hashlib.md5 with the usedforsecurity # keyword can just use that md5 definition as-is # See https://bugs.python.org/issue9216 + # + # TODO(alee) Remove this wrapper when the minimum python version is bumped + # to 3.9 (which is the first upstream version to support this keyword) + # See https://docs.python.org/3.9/library/hashlib.html md5 = hashlib.md5 except TypeError: def md5(string=b'', usedforsecurity=True):