Comment 1 for bug 1886984

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to oslo.policy (master)

Reviewed: https://review.opendev.org/740068
Committed: https://git.openstack.org/cgit/openstack/oslo.policy/commit/?id=65b96ab006f8ab8183afd509823110345460e149
Submitter: Zuul
Branch: master

commit 65b96ab006f8ab8183afd509823110345460e149
Author: Ben Nemec <email address hidden>
Date: Wed Jul 8 16:55:10 2020 +0000

    Don't deepcopy objects before mask_dict_password

    As far as I can tell, mask_dict_password does not modify the object
    passed in to it[0]. As such, this deepcopy only adds an unnecessary
    requirement on the policy objects that makes it possible for a call
    to fail in a different way when debug logging is enabled. Since this
    is pretty terrible, let's get rid of it.

    Change-Id: I34eace9806e6ed7c9c6206a34f55debc0c20bac6
    Closes-Bug: 1886984
    0: https://github.com/openstack/oslo.utils/blob/4fe75b7e1bd3144282f107ce7cb61880257c7c1e/oslo_utils/strutils.py#L349