Print debug-level log for DNS lookups when disabled

Signed-off-by: Matheus Pimenta <matheuscscp@gmail.com>

pkg/action/action.go

@@ -118,28 +118,22 @@ func (cfg *Configuration) renderResources(ch *chart.Chart, values chartutil.Valu
 		}
 	}
 
-	var files map[string]string
-	var err2 error
-
 	// A `helm template` should not talk to the remote cluster. However, commands with the flag
 	//`--dry-run` with the value of `false`, `none`, or `server` should try to interact with the cluster.
 	// It may break in interesting and exotic ways because other data (e.g. discovery) is mocked.
+	var e engine.Engine
 	if interactWithRemote && cfg.RESTClientGetter != nil {
 		restConfig, err := cfg.RESTClientGetter.ToRESTConfig()
 		if err != nil {
 			return hs, b, "", err
 		}
-		e := engine.New(restConfig)
+		e = engine.New(restConfig)
-		e.EnableDNS = enableDNS
-		files, err2 = e.Render(ch, values)
-	} else {
-		var e engine.Engine
-		e.EnableDNS = enableDNS
-		files, err2 = e.Render(ch, values)
 	}
-
+	e.EnableDNS = enableDNS
-	if err2 != nil {
+	e.Log = cfg.Log
-		return hs, b, "", err2
+	files, err := e.Render(ch, values)
+	if err != nil {
+		return hs, b, "", err
 	}
 
 	// NOTES.txt gets rendered like all the other files, but because it's not a hook nor a resource,

pkg/engine/engine.go

@@ -44,6 +44,8 @@ type Engine struct {
 	config *rest.Config
 	// EnableDNS tells the engine to allow DNS lookups when rendering templates
 	EnableDNS bool
+	// Log is used for printing logs.
+	Log func(string, ...interface{})
 }
 
 // New creates a new instance of Engine using the passed in rest config.
@@ -202,6 +204,9 @@ func (e Engine) initFunMap(t *template.Template, referenceTpls map[string]render
 	// an empty string.
 	if !e.EnableDNS {
 		funcMap["getHostByName"] = func(name string) string {
+			if e.Log != nil {
+				e.Log("DNS lookup is disabled by default. If you understand the security implications, check the documentation for how to enable DNS lookups in templates")
+			}
 			return ""
 		}
 	}