reverting tolowercase change

Author: jikuma

Tasks/Docker/Strings/resources.resjson/en-US/resources.resjson

@@ -23,7 +23,7 @@
   "loc.input.label.context": "Build Context",
   "loc.input.help.context": "Path to the build context.",
   "loc.input.label.imageName": "Image Name",
-  "loc.input.help.imageName": "Name of the Docker image to build, push or run. The name will be converted to lowercase as Docker supports only lowercase for image names.",
+  "loc.input.help.imageName": "Name of the Docker image to build, push or run.",
   "loc.input.label.qualifyImageName": "Qualify Image Name",
   "loc.input.help.qualifyImageName": "Qualify the image name with the Docker registry connection's hostname if not otherwise specified.",
   "loc.input.label.additionalImageTags": "Additional Image Tags",

Tasks/Docker/containerbuild.ts

@@ -27,17 +27,6 @@ function findDockerFile(dockerfilepath : string) : string {
     }
 }
 
-function changeImageNameToLowerCase(imageName: string): string {
-    var imageNameArray = imageName.split(":");
-
-    if(imageNameArray.length > 0)
-    {
-        imageName = imageNameArray[0].toLowerCase() + ":" +  imageNameArray.slice(1).join(":");
-    }
-
-    return imageName;
-} 
-
 export function run(connection: ContainerConnection): any {
     var command = connection.createCommand();
     command.arg("build");
@@ -56,7 +45,7 @@ export function run(connection: ContainerConnection): any {
     });
 
     var imageName = tl.getInput("imageName", true);
-    imageName = changeImageNameToLowerCase(imageName);
+    imageName = imageUtils.changeDefaultImageNameToLowerCase(imageName);
     var qualifyImageName = tl.getBoolInput("qualifyImageName");
     if (qualifyImageName) {
         imageName = connection.qualifyImageName(imageName);

Tasks/Docker/containerimageutils.ts

@@ -1,4 +1,5 @@
 "use strict";
+import * as tl from "vsts-task-lib/task";
 
 export function hasRegistryComponent(imageName: string): boolean {
     var periodIndex = imageName.indexOf("."),
@@ -18,3 +19,20 @@ export function imageNameWithoutTag(imageName: string): string {
     endIndex = imageName.indexOf(":", endIndex);
     return endIndex < 0 ? imageName : imageName.substr(0, endIndex);
 }
+
+export function changeDefaultImageNameToLowerCase(imageName: string): string {
+    
+    if(imageName === getDefaultImage())
+    {
+        return imageName.toLowerCase();
+    }
+    
+    return imageName;
+}
+
+function getDefaultImage() : string {
+    var repositoryName = tl.getVariable('build.repository.name');
+    var buildId = tl.getVariable('build.buildId');
+
+    return repositoryName + ":" + buildId;
+}

Tasks/Docker/task.json

@@ -115,7 +115,7 @@
             "defaultValue": "$(Build.Repository.Name):$(Build.BuildId)",
             "required": true,
             "visibleRule": "action = Build an image || action = Push an image || action = Run an image",
-            "helpMarkDown": "Name of the Docker image to build, push or run. The name will be converted to lowercase as Docker supports only lowercase for image names."
+            "helpMarkDown": "Name of the Docker image to build, push or run."
         },
         {
             "name": "qualifyImageName",