From 214251ff9aa0864dd08ed5ec551d990cd93a14eb Mon Sep 17 00:00:00 2001 From: John Eikenberry Date: Fri, 22 Jan 2021 12:30:48 -0800 Subject: [PATCH] porting vault fix from consul-template Fixes issue with vault's API returning pointer types for data field. Requires a nil check. Consul-template PR with more info. https://github.com/hashicorp/consul-template/pull/1447 --- internal/dependency/vault_common.go | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/internal/dependency/vault_common.go b/internal/dependency/vault_common.go index d43a358..d14a24f 100644 --- a/internal/dependency/vault_common.go +++ b/internal/dependency/vault_common.go @@ -1,6 +1,7 @@ package dependency import ( + "fmt" "math/rand" "path" "strings" @@ -258,6 +259,9 @@ func isKVv2(client *api.Client, path string) (string, bool, error) { if err != nil { return "", false, err } + if secret == nil { + return "", false, fmt.Errorf("secret at path %s does not exist", path) + } var mountPath string if mountPathRaw, ok := secret.Data["path"]; ok { mountPath = mountPathRaw.(string)