-
Notifications
You must be signed in to change notification settings - Fork 41
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
Enable custom SecretManager
s
#1809
Labels
Milestone
Comments
Hi @DeppLearning, it was closed because it will not be addressed on 1.0 and is already being implemented on 2.0. |
Oh I missed that, thank you for the clarification |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
This issue still persists: #407 . The closing PR #462 mentioned the issue of using custom secret managers wasn't resolved yet, still the issue was closed.
Motivation
The available variants in the
SecretManager
enum are not enough, and it would help adoption if there would be a generic api that others can implement without needing to fork.Requirements
Open questions (optional)
Are you planning to do it yourself in a pull request?
Maybe
The text was updated successfully, but these errors were encountered: