fix:[CI-13562]:improve error message for anonymous connector used (#34)

* fix:[CI-12799]:improve error message for anonymous connector used

* fix:[CI-13562]:improve error message for anonymous connector used

fix:[CI-13562]:typo

* fix:[CI-13562]: add error in command login too

* check for registry empty

Author: Aishwarya-Lad

docker.go

@@ -178,7 +178,13 @@ func (p Plugin) Exec() error {
 
 	// add base image docker credentials to the existing config file, else create new
 	// instead of writing to config file directly, using docker's login func
-	if p.BaseImagePassword != "" {
+	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.")
+		}
+		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.")
+		}
 		var baseConnectorLogin Login
 		baseConnectorLogin.Registry = p.BaseImageRegistry
 		baseConnectorLogin.Username = p.BaseImageUsername