Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding support of common security cipher module for encryption and decryption of a passkey #17201

Open
wants to merge 23 commits into
base: master
Choose a base branch
from
Open

Adding support of common security cipher module for encryption and decryption of a passkey #17201

Changes from 1 commit
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
be19898
Added a common encrypt/decrypt module w.r.t. HLD:TACACSPLUS_PASSKEY_E…
nmoray Nov 16, 2023
05e0e46
Merge branch 'master' of https://github.com/nmoray/sonic-buildimage i…
nmoray Nov 16, 2023
e681d60
Added test cases to test security cipher module
nmoray Nov 16, 2023
bc02ccf
Corrected typo
nmoray Nov 16, 2023
883fe3f
Addressed comments and fixed AUT build errors
nmoray Nov 17, 2023
e917f4d
Fixed build issues
nmoray Nov 20, 2023
4b3ebbd
Fixed AUT issues
nmoray Nov 22, 2023
dc67537
Fixed indentation issue
nmoray Nov 22, 2023
075f45e
Fixed build issues
nmoray Nov 22, 2023
6089def
Fixed build issues
nmoray Nov 22, 2023
4efb99d
Added support of mock class in existing AUT
nmoray Nov 22, 2023
5a2ef2f
fixed build issues
nmoray Nov 23, 2023
5c0455b
Fixed build issues
nmoray Nov 23, 2023
2079558
fixed build issues
nmoray Nov 23, 2023
b7f4f1d
Fixed build issues
nmoray Nov 23, 2023
542754a
fixed build issues
nmoray Nov 23, 2023
382690e
fixed build issues
nmoray Nov 23, 2023
494879b
Updated passkey leaf length to work with encrypted string too
nmoray Nov 24, 2023
014a0d4
Added a delete function for removing the footprint of cipher_pass fil…
nmoray Feb 1, 2024
b3babbd
Change the access permission of cipher_pass file from 644 to 640 (-rw…
nmoray Feb 1, 2024
470550f
updated scripts to mock chmod
nmoray Feb 1, 2024
167e820
Removed unwated debugs
nmoray Feb 2, 2024
7fc4c8d
renamed class name to master_key_mgr
nmoray Apr 11, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 4 additions & 2 deletions src/sonic-py-common/tests/test_security_cipher.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,8 @@

class TestSecurityCipher(object):
def test_passkey_encryption(self):
with mock.patch("sonic_py_common.security_cipher.ConfigDBConnector", new=ConfigDBConnector):
with mock.patch("sonic_py_common.security_cipher.ConfigDBConnector", new=ConfigDBConnector), \
mock.patch("builtins.open", mock_open()) as mock_file:
temp = security_cipher()
nmoray marked this conversation as resolved.
Show resolved Hide resolved

# Use patch to replace the built-in 'open' function with a mock
Expand All @@ -23,7 +24,8 @@ def test_passkey_encryption(self):
assert err == None

def test_passkey_decryption(self):
with mock.patch("sonic_py_common.security_cipher.ConfigDBConnector", new=ConfigDBConnector):
with mock.patch("sonic_py_common.security_cipher.ConfigDBConnector", new=ConfigDBConnector), \
mock.patch("builtins.open", mock_open()) as mock_file:
temp = security_cipher()
nmoray marked this conversation as resolved.
Show resolved Hide resolved
nmoray marked this conversation as resolved.
Show resolved Hide resolved

# Use patch to replace the built-in 'open' function with a mock
Expand Down
Loading