-
Notifications
You must be signed in to change notification settings - Fork 1
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
Share secrets by username #61
Conversation
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.
pending test
Changes:
|
client.go
Outdated
if err != nil { | ||
return nil, err | ||
} | ||
ownerClientID, err := uuid.Parse(c.StorageClient.ClientID) | ||
if err != nil { | ||
return nil, err |
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.
remember that a key reason for this change is to return an informative error message, suggest
return nil, errors.New(fmt.Sprintf("CreateSecret: Client ID must be a valid UUID, got %s", c.StorageClient.ClientID))
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.
got it! will update these error messages throughout.
Share secrets with identities by username, with a test.
DecryptTextSecret was updated to use the group eak, so that it can decrypt secrets that are owned or shared with the identity.