-
Notifications
You must be signed in to change notification settings - Fork 5
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
feat(secrets-vault): Add Secrets Server implementation for Vault #1
Conversation
6257933
to
ecc3578
Compare
2bc56e6
to
17a4d31
Compare
054dd9b
to
cc87c41
Compare
7ff3687
to
f85d7e3
Compare
a40bf32
to
32eb611
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mostly tiny nits, a couple questions
a909a94
to
e8034b3
Compare
026651e
to
5fd018b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, secrets-types is published now so that can be integrated here or as a followup
secrets/secrets-vault/Cargo.toml
Outdated
# TODO: update this to use the published crate once it exists. | ||
wasmcloud-secrets-types = { version = "0.1.0", git = "https://github.com/wasmcloud/wasmCloud.git", branch = "main" } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ready now!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will also need secrets-client
published so that there isn't a conflict between the types used by the git-based secrets-client
and the published secrets-type
crate.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tag pushed 😉 Should be ready here in just a bit
Co-authored-by: Brooks Townsend <brooksmtownsend@gmail.com> Signed-off-by: Joonas Bergius <joonas@cosmonic.com>
5fd018b
to
29e9c9f
Compare
Feature or Problem
This adds a Secrets RFC Secrets Server implementation for HashiCorp Vault
Related Issues
Release Information
Consumer Impact
Testing
Unit Test(s)
Acceptance or Integration
Manual Verification