Merge pull request #35 from Aishwarya-Lad/CI-13924

fix:[CI-13924]: print validation instead of raising error
drone-plugins · Aug 21, 2024 · 387a8e4 · 387a8e4
2 parents 62c3055 + 16924a0
commit 387a8e4
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/docker.go b/docker.go
@@ -180,10 +180,10 @@ func (p Plugin) Exec() error {
 	// instead of writing to config file directly, using docker's login func
 	if p.BaseImageRegistry != "" {
 		if p.BaseImageUsername == "" {
-			return fmt.Errorf("Username cannot be empty. The base image connector requires authenticated access. Please either use an authenticated connector, or remove the base image connector.")
+			fmt.Printf("Username cannot be empty. The base image connector requires authenticated access. Please either use an authenticated connector, or remove the base image connector.")
 		}
 		if p.BaseImagePassword == "" {
-			return fmt.Errorf("Password cannot be empty. The base image connector requires authenticated access. Please either use an authenticated connector, or remove the base image connector.")
+			fmt.Printf("Password cannot be empty. The base image connector requires authenticated access. Please either use an authenticated connector, or remove the base image connector.")
 		}
 		var baseConnectorLogin Login
 		baseConnectorLogin.Registry = p.BaseImageRegistry