Updated Agent code to handling '+' sign in file path for build configs (#5175)

Co-authored-by: Sanju Yadav <sanjuyadav@microsoft.com>

Author: sanjuyadav24

src/Agent.Worker/TaskManager.cs

@@ -182,7 +182,7 @@ private async Task DownloadAsync(IExecutionContext executionContext, Pipelines.T
                 return;
             }
 
-            String taskZipPath = Path.Combine(HostContext.GetDirectory(WellKnownDirectory.TaskZips), $"{task.Name}_{task.Id}_{task.Version}.zip");
+            String taskZipPath = Path.Combine(HostContext.GetDirectory(WellKnownDirectory.TaskZips), $"{task.Name}_{task.Id}_{NormalizeTaskVersion(task)}.zip");
             if (alwaysExtractTask && File.Exists(taskZipPath))
             {
                 executionContext.Debug($"Task '{task.Name}' already downloaded at '{taskZipPath}'.");
@@ -472,7 +472,13 @@ private string GetDirectory(Pipelines.TaskStepDefinitionReference task)
             return Path.Combine(
                 HostContext.GetDirectory(WellKnownDirectory.Tasks),
                 $"{task.Name}_{task.Id}",
-                task.Version);
+                NormalizeTaskVersion(task));
+        }
+
+        private string NormalizeTaskVersion(Pipelines.TaskStepDefinitionReference task) 
+        {
+            ArgUtil.NotNullOrEmpty(task.Version, nameof(task.Version));
+            return task.Version.Replace("+", "_");
         }
 
         private string GetTaskZipPath(Pipelines.TaskStepDefinitionReference task)
@@ -482,7 +488,7 @@ private string GetTaskZipPath(Pipelines.TaskStepDefinitionReference task)
             ArgUtil.NotNullOrEmpty(task.Version, nameof(task.Version));
             return Path.Combine(
                 HostContext.GetDirectory(WellKnownDirectory.TaskZips),
-                $"{task.Name}_{task.Id}_{task.Version}.zip"); // TODO: Move to shared string.
+                $"{task.Name}_{task.Id}_{NormalizeTaskVersion(task)}.zip"); // TODO: Move to shared string.
         }
 
         private Definition GetTaskDefiniton(Pipelines.TaskStep task)

src/Microsoft.VisualStudio.Services.Agent/TaskServer.cs

@@ -69,7 +69,14 @@ private void CheckConnection()
         public Task<Stream> GetTaskContentZipAsync(Guid taskId, TaskVersion taskVersion, CancellationToken token)
         {
             CheckConnection();
-            return _taskAgentClient.GetTaskContentZipAsync(taskId, taskVersion, cancellationToken: token);
+            return _taskAgentClient.GetTaskContentZipAsync(
+                taskId,
+                versionString: taskVersion,
+                visibility: null,
+                scopeLocal: null,
+                userState: null,
+                cancellationToken: token
+            );
         }
 
         public async Task<bool> TaskDefinitionEndpointExist()