yahoo-eng-team team mailing list archive

[OpenStack Infra <2081732@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://review.opendev.org/c/openstack/octavia/+/931150
Committed: https://opendev.org/openstack/octavia/commit/119b14ed93b856bada54eaea9155f44b8b0629ff
Submitter: "Zuul (22348)"
Branch:    master

commit 119b14ed93b856bada54eaea9155f44b8b0629ff
Author: Takashi Kajinami <kajinamit@xxxxxxxxxxxxxxx>
Date:   Wed Oct 2 18:32:34 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: Icf9f8086e7f413247532d3f234a036b2474b7ef3


** Changed in: octavia
       Status: New => 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:
  New
Status in keystonemiddleware:
  In Progress
Status in OpenStack Compute (nova):
  In Progress
Status in octavia:
  Fix Released
Status in oslo.utils:
  Fix Released
Status in osprofiler:
  In Progress

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