Vaidate full image name passed in for '--init-image'

fnproject · Sep 25, 2018 · 6b035dc · 6b035dc
1 parent 396c29f
commit 6b035dc
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/commands/init.go b/commands/init.go
@@ -208,7 +208,7 @@ func (a *initFnCmd) init(c *cli.Context) error {
 		a.triggerType = strings.ToLower(a.triggerType)
 		ok := validateTriggerType(a.triggerType)
 		if !ok {
-			return fmt.Errorf("Init does not support the trigger type '%s'.\n", a.triggerType, " Permitted values are 'http'.")
+			return fmt.Errorf("Init does not support the trigger type '%s'.\n Permitted values are 'http'.", a.triggerType)
 		}
 
 		trig := make([]common.Trigger, 1)
@@ -246,6 +246,10 @@ func (a *initFnCmd) init(c *cli.Context) error {
 	a.ff.Schema_version = common.LatestYamlVersion
 
 	if initImage != "" {
+		err = common.ValidateFullImageName(initImage)
+		if err != nil {
+			return err
+		}
 
 		err = runInitImage(initImage, a)
 		if err != nil {

diff --git a/common/common.go b/common/common.go
@@ -503,7 +503,6 @@ func DockerPushV20180708(ff *FuncFileV20180708) error {
 // remember that private registries must be supported here
 func ValidateFullImageName(n string) error {
 	parts := strings.Split(n, "/")
-	fmt.Println("Parts: ", parts)
 	if len(parts) < 2 {
 		return errors.New("image name must have a dockerhub owner or private registry. Be sure to set FN_REGISTRY env var, pass in --registry or configure your context file")