From ac234771c79db3e54dcc306adca249df72f386ae Mon Sep 17 00:00:00 2001 From: max furman Date: Thu, 29 Aug 2019 10:49:52 -0700 Subject: [PATCH] Remove unknown provisioner WARNning and leave TODO --- authority/provisioner/provisioner.go | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/authority/provisioner/provisioner.go b/authority/provisioner/provisioner.go index c6c53455..3118e6b0 100644 --- a/authority/provisioner/provisioner.go +++ b/authority/provisioner/provisioner.go @@ -3,7 +3,6 @@ package provisioner import ( "crypto/x509" "encoding/json" - "log" "net/url" "strings" @@ -155,11 +154,11 @@ func (l *List) UnmarshalJSON(data []byte) error { // compiled with a version of smallstep/certificates that does not // support a specific provisioner type. If we don't skip unknown // provisioners, a client encountering an unknown provisioner will - // break. Rather than break the client, we skip the provisioner but + // break. Rather than break the client, we skip the provisioner. + // TODO: accept a pluggable logger (depending on client) that can // warn the user that an unknown provisioner was found and suggest // that the user update their client's dependency on // step/certificates and recompile. - log.Printf("[WARN] Unknown provisioner of type '%s' found. Please update your client.", typ.Type) continue } if err := json.Unmarshal(data, p); err != nil {