← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 2081732] Re: oslo_utils.secretutils.constant_time_compare is redundant

 

Reviewed:  https://review.opendev.org/c/openstack/osprofiler/+/931151
Committed: https://opendev.org/openstack/osprofiler/commit/c1faa8478c2c821c268ddd619f9c1149b9f5d7b2
Submitter: "Zuul (22348)"
Branch:    master

commit c1faa8478c2c821c268ddd619f9c1149b9f5d7b2
Author: Takashi Kajinami <kajinamit@xxxxxxxxxxxxxxx>
Date:   Wed Oct 2 18:34:03 2024 +0900

    Replace deprecated constant_time_compare
    
    The method is being deprecated now[1].
    
    [1] https://review.opendev.org/c/openstack/oslo.utils/+/930198
    
    Closes-Bug: #2081732
    Change-Id: Icd707efef0375f4938cd60376b892430c36083c7


** Changed in: osprofiler
       Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/2081732

Title:
  oslo_utils.secretutils.constant_time_compare is redundant

Status in Ceilometer:
  Fix Released
Status in keystonemiddleware:
  Fix Released
Status in OpenStack Compute (nova):
  In Progress
Status in octavia:
  Fix Released
Status in oslo.utils:
  Fix Released
Status in osprofiler:
  Fix Released

Bug description:
  The constant_time_compare function is equivalent to
  hmac.compare_digest in Python 3, because the constant_time_compare
  function has been available since Python 3.3 .

  [1] https://docs.python.org/3/library/hmac.html#hmac.compare_digest

  We can get rid of the redundant wrapper and use the built-in
  implementation instead.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ceilometer/+bug/2081732/+subscriptions