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.

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

Add Secure OS Storage Option #624

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

Conversation

EndlessTrax
Copy link

This PR introduced an option for the user to use a secure storage option for configuration file to avoid KSM configuration files being store in a plaintext file.

The storage interface makes use of the windows-credential-manager-utility or the linux-keyring-utility to intact with the operating systems native secrets platform (i.e. Windows Credential Manager or Linux Keyring).

In order to use the SecureOSStorage the user is expected to set a KSM_CONFIG_SECURE environment variable to True.

Optionally, the user may set KSM_CONFIG_EXE_PATH to the path of the utility executable (listed above). By default, SecureOSStorage will check the users PATH for the utility. By setting the environment variable, users have the option to run the executable directly without it being part of PATH.

@EndlessTrax EndlessTrax force-pushed the add-secure-config-storage branch from bb1d939 to a035b6e Compare July 26, 2024 15:27
@EndlessTrax EndlessTrax force-pushed the add-secure-config-storage branch from c54a667 to a25d8a7 Compare July 26, 2024 15:37
maksimu
maksimu previously approved these changes Jul 26, 2024
@EndlessTrax EndlessTrax requested a review from maksimu July 30, 2024 18:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants