keep
keep copied to clipboard
[🔨 Enhancement]: Secrets Manager - add error handling and exceptions
Current:
- BaseSecretManager - read_secret, write_secret as abstract methods
- Every deriving manager implements those functions
- No error handling
Expected Behaviour
- BaseSecretManager - _read_secret, _write_secret as abstract methods
- BaseSecretManager - read_secret and write_secret which uses the
_methods with try/except - Every deriving manager implements error handling with at least PermissionError
i can work on this
@shahargl I think it might "hide" some problems if we do it that way (e.g., when something is calling read_secret and expects the exception, or something like that) I'm closing for now and will re-open if it raises in any way