Skip to content

Commit fbb7d30

Browse files
authored
Fix download of fileshare artifact when location contains spaces. (#6053)
* Fix download of fileshare artifact when location contains spaces. * bump up artifact-engine version
1 parent 89428bf commit fbb7d30

File tree

10 files changed

+11
-11
lines changed

10 files changed

+11
-11
lines changed

Tasks/AzureVmssDeployment/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,6 @@
33
"main": "main.js",
44
"dependencies": {
55
"vsts-task-lib": "2.1.0",
6-
"artifact-engine": "0.1.13"
6+
"artifact-engine": "0.1.14"
77
}
88
}

Tasks/Common/azure-blobstorage-artifactProvider/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
"homepage": "https://github.com/Microsoft/vsts-tasks#readme",
1515
"dependencies": {
1616
"q": "1.4.1",
17-
"artifact-engine": "0.1.13",
17+
"artifact-engine": "0.1.14",
1818
"azure-storage": "2.2.1",
1919
"vsts-task-lib": "2.1.0"
2020
}

Tasks/DownloadBuildArtifacts/main.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ function publishEvent(feature, properties: any): void {
4444
else {
4545
if (feature === 'reliability') {
4646
let reliabilityData = properties;
47-
telemetry = "##vso[task.logissue type=error;code=" + reliabilityData.issueType + ";agentVersion=" + tl.getVariable('Agent.Version') + ";taskId=" + area + "-" + taskJson.version + ";]" + reliabilityData.errorMessage
47+
telemetry = "##vso[task.logissue type=error;code=" + reliabilityData.issueType + ";agentVersion=" + tl.getVariable('Agent.Version') + ";taskId=" + area + "-" + JSON.stringify(taskJson.version) + ";]" + reliabilityData.errorMessage
4848
}
4949
}
5050
console.log(telemetry);;
@@ -163,7 +163,7 @@ async function main(): Promise<void> {
163163
}
164164
else if (artifact.resource.type.toLowerCase() === "filepath") {
165165
let downloader = new engine.ArtifactEngine();
166-
let downloadUrl = artifact.resource.downloadUrl.replace("file:", "");
166+
let downloadUrl = decodeURI(artifact.resource.downloadUrl.replace("file:", ""));
167167
let artifactLocation = downloadUrl + '/' + artifact.name;
168168
if (!fs.existsSync(artifactLocation)) {
169169
console.log(tl.loc("ArtifactNameDirectoryNotFound", artifactLocation, downloadUrl));

Tasks/DownloadBuildArtifacts/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,6 @@
1919
"dependencies": {
2020
"vsts-task-lib": "2.1.0",
2121
"vso-node-api": "^6.2.8-preview",
22-
"artifact-engine": "0.1.13"
22+
"artifact-engine": "0.1.14"
2323
}
2424
}

Tasks/DownloadBuildArtifacts/task.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
"version": {
1111
"Major": 0,
1212
"Minor": 127,
13-
"Patch": 4
13+
"Patch": 5
1414
},
1515
"groups": [
1616
{

Tasks/DownloadBuildArtifacts/task.loc.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
"version": {
1111
"Major": 0,
1212
"Minor": 127,
13-
"Patch": 4
13+
"Patch": 5
1414
},
1515
"groups": [
1616
{

Tasks/JenkinsDownloadArtifacts/jenkinsdownloadartifacts.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ function publishEvent(feature, properties: any): void {
6868
else {
6969
if (feature === 'reliability') {
7070
let reliabilityData = properties;
71-
telemetry = "##vso[task.logissue type=error;code=" + reliabilityData.issueType + ";agentVersion=" + tl.getVariable('Agent.Version') + ";taskId=" + area + "-" + taskJson.version + ";]" + reliabilityData.errorMessage
71+
telemetry = "##vso[task.logissue type=error;code=" + reliabilityData.issueType + ";agentVersion=" + tl.getVariable('Agent.Version') + ";taskId=" + area + "-" + JSON.stringify(taskJson.version) + ";]" + reliabilityData.errorMessage
7272
}
7373
}
7474
console.log(telemetry);;

Tasks/JenkinsDownloadArtifacts/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
"author": "Microsoft Corporation",
99
"license": "MIT",
1010
"dependencies": {
11-
"artifact-engine": "0.1.13",
11+
"artifact-engine": "0.1.14",
1212
"azure-arm-rest": "file:../../_build/Tasks/Common/azure-arm-rest",
1313
"azure-blobstorage-artifactProvider": "file:../../_build/Tasks/Common/azure-blobstorage-artifactProvider",
1414
"request": "2.74.0",

Tasks/JenkinsDownloadArtifacts/task.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818
"version": {
1919
"Major": 1,
2020
"Minor": 127,
21-
"Patch": 3
21+
"Patch": 4
2222
},
2323
"groups": [
2424
{

Tasks/JenkinsDownloadArtifacts/task.loc.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818
"version": {
1919
"Major": 1,
2020
"Minor": 127,
21-
"Patch": 3
21+
"Patch": 4
2222
},
2323
"groups": [
2424
{

0 commit comments

Comments
 (0)