Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Herman Slatman <hslatman@users.noreply.github.com>
  • Loading branch information
maraino and hslatman authored Dec 4, 2023
1 parent 01dcb48 commit f7a9002
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions cmd/root.go
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@ func openKMS(ctx context.Context, kuri string) (apiv1.KeyManager, error) {

// changeURI adds extra parameters to the given uri.
//
// If the given values are already in the rawuri, they will take preference.
// If the given values are already in the rawuri, the new values will take preference.
func changeURI(rawuri string, values url.Values) (string, error) {
u, err := uri.Parse(rawuri)
if err != nil {
Expand Down Expand Up @@ -128,7 +128,7 @@ func ensureSchemePrefix(kuri string) string {
}

// getURIAndNameForFS returns the kuri and name to be used by a KMS FS. If TPM
// KMS is used it changes the kuri to add the default storage directory.
// KMS is used, it changes the kuri to add the default storage directory.
//
// If a storage-directory is already in the kuri, this will take preference.
func getURIAndNameForFS(kuri, name string) (string, string, error) {
Expand Down

0 comments on commit f7a9002

Please sign in to comment.