From e1cea08775122e4ca1f13fe4d161f9018324ee0a Mon Sep 17 00:00:00 2001 From: Charlie Kim Date: Tue, 1 Mar 2022 14:37:18 -0500 Subject: [PATCH 1/2] request data can be empty string --- aos/utils.py | 2 +- setup.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/aos/utils.py b/aos/utils.py index 3dd3d96..f747f17 100644 --- a/aos/utils.py +++ b/aos/utils.py @@ -8,7 +8,7 @@ def redacted(d): - if d is None: + if d is None or d == '': return d h = d.copy() diff --git a/setup.py b/setup.py index c96da54..972e2b0 100644 --- a/setup.py +++ b/setup.py @@ -9,7 +9,7 @@ NAME = "aos-api-client" -VERSION = '0.1.9' +VERSION = '0.1.10' REQUIRES = (["requests==2.24.0"],) From f7f596b79ec38f19310a71ffcc79f53f850a80e3 Mon Sep 17 00:00:00 2001 From: Charlie Kim Date: Tue, 1 Mar 2022 15:01:49 -0500 Subject: [PATCH 2/2] add test_utils --- tests/test_utils.py | 28 ++++++++++++++++++++++++++++ 1 file changed, 28 insertions(+) create mode 100644 tests/test_utils.py diff --git a/tests/test_utils.py b/tests/test_utils.py new file mode 100644 index 0000000..0558de4 --- /dev/null +++ b/tests/test_utils.py @@ -0,0 +1,28 @@ + +# Copyright 2020-present, Apstra, Inc. All rights reserved. +# +# This source code is licensed under End User License Agreement found in the +# LICENSE file at http://www.apstra.com/eula +# pylint: disable=redefined-outer-name + +from aos.utils import redacted + + +def test_redacted_null(): + assert redacted(None) is None + + +def test_redacted_empty(): + assert redacted('') == '' + + +def test_redacted_sensitive(): + for sensitive in ["password", "token", "AuthToken"]: + d = { + 'usual': 'usual data', + sensitive: '{sensitive} data', + } + assert redacted(d) == { + 'usual': 'usual data', + sensitive: '', + }