Skip to content

Commit

Permalink
Hide PowerShell tasks extra log under debug mode (#19556)
Browse files Browse the repository at this point in the history
* Make args sanitization log as debug

* Bump WindowsMachineFileCopy tasks

* Bump AzureFileCopy tasks

* Bump powershell task
  • Loading branch information
KonstantinTyukalov authored Feb 21, 2024
1 parent b4a2bbd commit 9172b33
Show file tree
Hide file tree
Showing 43 changed files with 92 additions and 93 deletions.
2 changes: 1 addition & 1 deletion Tasks/AzureFileCopyV1/task.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"author": "Microsoft Corporation",
"version": {
"Major": 1,
"Minor": 234,
"Minor": 236,
"Patch": 0
},
"demands": [
Expand Down
2 changes: 1 addition & 1 deletion Tasks/AzureFileCopyV1/task.loc.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"author": "Microsoft Corporation",
"version": {
"Major": 1,
"Minor": 234,
"Minor": 236,
"Patch": 0
},
"demands": [
Expand Down
2 changes: 1 addition & 1 deletion Tasks/AzureFileCopyV2/task.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"author": "Microsoft Corporation",
"version": {
"Major": 2,
"Minor": 234,
"Minor": 236,
"Patch": 0
},
"demands": [
Expand Down
2 changes: 1 addition & 1 deletion Tasks/AzureFileCopyV2/task.loc.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"author": "Microsoft Corporation",
"version": {
"Major": 2,
"Minor": 234,
"Minor": 236,
"Patch": 0
},
"demands": [
Expand Down
2 changes: 1 addition & 1 deletion Tasks/AzureFileCopyV3/task.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"author": "Microsoft Corporation",
"version": {
"Major": 3,
"Minor": 234,
"Minor": 236,
"Patch": 0
},
"demands": [
Expand Down
2 changes: 1 addition & 1 deletion Tasks/AzureFileCopyV3/task.loc.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"author": "Microsoft Corporation",
"version": {
"Major": 3,
"Minor": 234,
"Minor": 236,
"Patch": 0
},
"demands": [
Expand Down
2 changes: 1 addition & 1 deletion Tasks/AzureFileCopyV4/task.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"author": "Microsoft Corporation",
"version": {
"Major": 4,
"Minor": 234,
"Minor": 236,
"Patch": 0
},
"demands": [
Expand Down
2 changes: 1 addition & 1 deletion Tasks/AzureFileCopyV4/task.loc.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"author": "Microsoft Corporation",
"version": {
"Major": 4,
"Minor": 234,
"Minor": 236,
"Patch": 0
},
"demands": [
Expand Down
2 changes: 1 addition & 1 deletion Tasks/AzureFileCopyV5/task.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"author": "Microsoft Corporation",
"version": {
"Major": 5,
"Minor": 234,
"Minor": 236,
"Patch": 0
},
"demands": [
Expand Down
2 changes: 1 addition & 1 deletion Tasks/AzureFileCopyV5/task.loc.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"author": "Microsoft Corporation",
"version": {
"Major": 5,
"Minor": 234,
"Minor": 236,
"Patch": 0
},
"demands": [
Expand Down
2 changes: 1 addition & 1 deletion Tasks/Common/Sanitizer/ArgumentsSanitizer.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ function Protect-ScriptArguments([string]$inputArgs, [string]$taskName) {
$sanitizedArgs, $sanitizeTelemetry = Get-SanitizedArguments -InputArgs $expandedArgs

if ($sanitizedArgs -eq $inputArgs) {
Write-Host (Get-VstsLocString -Key 'PS_ScriptArgsNotSanitized');
Write-Debug 'Arguments passed sanitization without change.'
}
else {
if ($featureFlags.telemetry) {
Expand Down
3 changes: 1 addition & 2 deletions Tasks/Common/Sanitizer/module.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
{
"messages": {
"PS_ScriptArgsSanitized": "Detected characters in arguments that may not be executed correctly by the shell. Please escape special characters using backtick (`). More information is available here: https://aka.ms/ado/75787",
"PS_ScriptArgsNotSanitized": "Arguments passed sanitization without change."
"PS_ScriptArgsSanitized": "Detected characters in arguments that may not be executed correctly by the shell. Please escape special characters using backtick (`). More information is available here: https://aka.ms/ado/75787"
}
}
2 changes: 1 addition & 1 deletion Tasks/PowerShellV2/task.json
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
"author": "Microsoft Corporation",
"version": {
"Major": 2,
"Minor": 235,
"Minor": 236,
"Patch": 0
},
"releaseNotes": "Script task consistency. Added support for macOS and Linux.",
Expand Down
2 changes: 1 addition & 1 deletion Tasks/PowerShellV2/task.loc.json
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
"author": "Microsoft Corporation",
"version": {
"Major": 2,
"Minor": 235,
"Minor": 236,
"Patch": 0
},
"releaseNotes": "ms-resource:loc.releaseNotes",
Expand Down
4 changes: 2 additions & 2 deletions Tasks/WindowsMachineFileCopyV1/task.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,8 @@
"author": "Microsoft Corporation",
"version": {
"Major": 1,
"Minor": 231,
"Patch": 1
"Minor": 236,
"Patch": 0
},
"minimumAgentVersion": "1.104.0",
"groups": [
Expand Down
4 changes: 2 additions & 2 deletions Tasks/WindowsMachineFileCopyV1/task.loc.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,8 @@
"author": "Microsoft Corporation",
"version": {
"Major": 1,
"Minor": 231,
"Patch": 1
"Minor": 236,
"Patch": 0
},
"minimumAgentVersion": "1.104.0",
"groups": [
Expand Down
4 changes: 2 additions & 2 deletions Tasks/WindowsMachineFileCopyV2/task.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,8 @@
"author": "Microsoft Corporation",
"version": {
"Major": 2,
"Minor": 231,
"Patch": 1
"Minor": 236,
"Patch": 0
},
"releaseNotes": "What's new in Version 2.0: <br/>&nbsp;&nbsp;Proxy support is being added. <br/>&nbsp;&nbsp; Removed support of legacy DTL machines.",
"minimumAgentVersion": "1.104.0",
Expand Down
4 changes: 2 additions & 2 deletions Tasks/WindowsMachineFileCopyV2/task.loc.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,8 @@
"author": "Microsoft Corporation",
"version": {
"Major": 2,
"Minor": 231,
"Patch": 1
"Minor": 236,
"Patch": 0
},
"releaseNotes": "ms-resource:loc.releaseNotes",
"minimumAgentVersion": "1.104.0",
Expand Down
4 changes: 2 additions & 2 deletions _generated/AzureFileCopyV2.versionmap.txt
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
Default|2.234.0
Node20_229_2|2.234.1
Default|2.236.0
Node20_229_2|2.236.1
6 changes: 3 additions & 3 deletions _generated/AzureFileCopyV2/task.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"author": "Microsoft Corporation",
"version": {
"Major": 2,
"Minor": 234,
"Minor": 236,
"Patch": 0
},
"demands": [
Expand Down Expand Up @@ -374,7 +374,7 @@
"ExpiredServicePrincipal": "Could not fetch access token for Azure. Verify if the Service Principal used is valid and not expired."
},
"_buildConfigMapping": {
"Default": "2.234.0",
"Node20_229_2": "2.234.1"
"Default": "2.236.0",
"Node20_229_2": "2.236.1"
}
}
6 changes: 3 additions & 3 deletions _generated/AzureFileCopyV2/task.loc.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"author": "Microsoft Corporation",
"version": {
"Major": 2,
"Minor": 234,
"Minor": 236,
"Patch": 0
},
"demands": [
Expand Down Expand Up @@ -374,7 +374,7 @@
"ExpiredServicePrincipal": "ms-resource:loc.messages.ExpiredServicePrincipal"
},
"_buildConfigMapping": {
"Default": "2.234.0",
"Node20_229_2": "2.234.1"
"Default": "2.236.0",
"Node20_229_2": "2.236.1"
}
}
6 changes: 3 additions & 3 deletions _generated/AzureFileCopyV2_Node20/task.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"author": "Microsoft Corporation",
"version": {
"Major": 2,
"Minor": 234,
"Minor": 236,
"Patch": 1
},
"demands": [
Expand Down Expand Up @@ -378,7 +378,7 @@
"ExpiredServicePrincipal": "Could not fetch access token for Azure. Verify if the Service Principal used is valid and not expired."
},
"_buildConfigMapping": {
"Default": "2.234.0",
"Node20_229_2": "2.234.1"
"Default": "2.236.0",
"Node20_229_2": "2.236.1"
}
}
6 changes: 3 additions & 3 deletions _generated/AzureFileCopyV2_Node20/task.loc.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"author": "Microsoft Corporation",
"version": {
"Major": 2,
"Minor": 234,
"Minor": 236,
"Patch": 1
},
"demands": [
Expand Down Expand Up @@ -378,7 +378,7 @@
"ExpiredServicePrincipal": "ms-resource:loc.messages.ExpiredServicePrincipal"
},
"_buildConfigMapping": {
"Default": "2.234.0",
"Node20_229_2": "2.234.1"
"Default": "2.236.0",
"Node20_229_2": "2.236.1"
}
}
4 changes: 2 additions & 2 deletions _generated/AzureFileCopyV3.versionmap.txt
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
Default|3.234.0
Node20_229_2|3.234.1
Default|3.236.0
Node20_229_2|3.236.1
6 changes: 3 additions & 3 deletions _generated/AzureFileCopyV3/task.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"author": "Microsoft Corporation",
"version": {
"Major": 3,
"Minor": 234,
"Minor": 236,
"Patch": 0
},
"demands": [
Expand Down Expand Up @@ -313,7 +313,7 @@
"ExpiredServicePrincipal": "Could not fetch access token for Azure. Verify if the Service Principal used is valid and not expired."
},
"_buildConfigMapping": {
"Default": "3.234.0",
"Node20_229_2": "3.234.1"
"Default": "3.236.0",
"Node20_229_2": "3.236.1"
}
}
6 changes: 3 additions & 3 deletions _generated/AzureFileCopyV3/task.loc.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"author": "Microsoft Corporation",
"version": {
"Major": 3,
"Minor": 234,
"Minor": 236,
"Patch": 0
},
"demands": [
Expand Down Expand Up @@ -313,7 +313,7 @@
"ExpiredServicePrincipal": "ms-resource:loc.messages.ExpiredServicePrincipal"
},
"_buildConfigMapping": {
"Default": "3.234.0",
"Node20_229_2": "3.234.1"
"Default": "3.236.0",
"Node20_229_2": "3.236.1"
}
}
6 changes: 3 additions & 3 deletions _generated/AzureFileCopyV3_Node20/task.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"author": "Microsoft Corporation",
"version": {
"Major": 3,
"Minor": 234,
"Minor": 236,
"Patch": 1
},
"demands": [
Expand Down Expand Up @@ -317,7 +317,7 @@
"ExpiredServicePrincipal": "Could not fetch access token for Azure. Verify if the Service Principal used is valid and not expired."
},
"_buildConfigMapping": {
"Default": "3.234.0",
"Node20_229_2": "3.234.1"
"Default": "3.236.0",
"Node20_229_2": "3.236.1"
}
}
6 changes: 3 additions & 3 deletions _generated/AzureFileCopyV3_Node20/task.loc.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"author": "Microsoft Corporation",
"version": {
"Major": 3,
"Minor": 234,
"Minor": 236,
"Patch": 1
},
"demands": [
Expand Down Expand Up @@ -317,7 +317,7 @@
"ExpiredServicePrincipal": "ms-resource:loc.messages.ExpiredServicePrincipal"
},
"_buildConfigMapping": {
"Default": "3.234.0",
"Node20_229_2": "3.234.1"
"Default": "3.236.0",
"Node20_229_2": "3.236.1"
}
}
4 changes: 2 additions & 2 deletions _generated/AzureFileCopyV4.versionmap.txt
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
Default|4.234.0
Node20_229_2|4.234.1
Default|4.236.0
Node20_229_2|4.236.1
6 changes: 3 additions & 3 deletions _generated/AzureFileCopyV4/task.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"author": "Microsoft Corporation",
"version": {
"Major": 4,
"Minor": 234,
"Minor": 236,
"Patch": 0
},
"demands": [
Expand Down Expand Up @@ -301,7 +301,7 @@
"ServicePrincipalError": "There was an error with the service principal used for the deployment."
},
"_buildConfigMapping": {
"Default": "4.234.0",
"Node20_229_2": "4.234.1"
"Default": "4.236.0",
"Node20_229_2": "4.236.1"
}
}
6 changes: 3 additions & 3 deletions _generated/AzureFileCopyV4/task.loc.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"author": "Microsoft Corporation",
"version": {
"Major": 4,
"Minor": 234,
"Minor": 236,
"Patch": 0
},
"demands": [
Expand Down Expand Up @@ -301,7 +301,7 @@
"ServicePrincipalError": "ms-resource:loc.messages.ServicePrincipalError"
},
"_buildConfigMapping": {
"Default": "4.234.0",
"Node20_229_2": "4.234.1"
"Default": "4.236.0",
"Node20_229_2": "4.236.1"
}
}
Loading

0 comments on commit 9172b33

Please sign in to comment.