Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update toolrunner.ts #922

Open
wants to merge 263 commits into
base: master
Choose a base branch
from
Open

Conversation

andrelvahter
Copy link

In case the command line arguments contain secrets or any other info that should not be printed out. We could use the same 'silent' flag to not add all the arguments to debug log.

ericsciple and others added 30 commits November 2, 2017 14:26
…ent-version

Add TFS 2017 Update 3 agent version.
Deep Dive on Building Custom Build or Deploy Tasks
* Add task.json schema file

* Allow any "connectedService:" type of input

* Added the "id" field
Properly escape property value to handle non string types
* Error on multiline secret

* v2.3.0
…t#325)

* Update dependencies to prevent warnings during npm install.

* Remove package-lock until we fully upgrade.
* Add supported commands to TaskLibAnswers

* Remove stray semicolon

* Typescript -> TypeScript
* Revert updates to dependency versions.

* Add package-lock.json.

* Remove root package lock.

* Try to get node_modules output.

* Cleanup.

* Fix make.ks.

* Add third party notice generator.

* Update docs.

* Add third party notice.

* Copy node_modules to _build.

* Copy ThirdPartyNotices.txt to _build directory.

* Prune dev dependencies in _build.

* Remove node_modules copy, shouldn't need to be there.

* Cleanup.

* Clean.

* Clean.
AndreyIvanov42 and others added 25 commits February 16, 2022 16:43
* Add HostException handler

* Update error handling for HostException

* Update vmImage for windows part

* Update way of processing HostException

* Change image to win 2019

* Fix naming
* Added new required functions. Return the value instead of the value or undefined.

getInputRequired
getPathInputRequired
getEndpointUrlRequired
getEndpointDataParameterRequired
getEndpointAuthorizationSchemeRequired
getEndpointAuthorizationParameterRequired

* Fix undefined in json file

* Simplify Required functions. Add tests.

Co-authored-by: Keith Banner <[email protected]>
* added formatted proxy URL
* version increment
Co-authored-by: microsoft-github-policy-service[bot] <77245923+microsoft-github-policy-service[bot]@users.noreply.github.com>
Co-authored-by: Konstantin Tyukalov <[email protected]>
…t#848)

* Update pipeline

* Update azure-pipelines.yml

* nodeversion as variable

Co-authored-by: Andrey Ivanov <[email protected]>
Co-authored-by: Andrey Ivanov <[email protected]>
…ted agent, or not. (microsoft#869)

* Added function getAgentMode

* Function getAgentMode added to documentation

* Fix of documentation
* Updated PSLIB_InvalidPattern0 to be more readable

* Update LegacyFindFunctions.ps1

Co-authored-by: Kirill Ivlev <[email protected]>
* Added localization pipeline and LocProject.json

* Removed en-US

* Update localize-pipeline.yml for Azure Pipelines

* Update localize-pipeline.yml for Azure Pipelines

* Made letter case consistent for languages

* LEGO: check in for Localization to temporary branch. (microsoft#703)

* LEGO: check in for Localization to temporary branch. (microsoft#714)

* LEGO: check in for Localization to temporary branch. (microsoft#720)

* Temp renaming

* Renamed localization files

* Applied enhancements for the localization pipeline (microsoft#733)

[skip ci]

* [Localization] Fixed localization pipeline issue with already localized strings replaced (microsoft#737)

* Localized file check-in by OneLocBuild Task: Build definition ID 10947: Build ID 14646607

Localized file check-in by OneLocBuild Task

* LEGO: check in for Localization to temporary branch. (microsoft#740)

* LEGO: check in for Localization to temporary branch. (microsoft#741)

* LEGO: check in for Localization to temporary branch. (microsoft#742)

* LEGO: check in for Localization to temporary branch. (microsoft#743)

* Temporary renamed files - to resolve conflicts

* Temporary renamed

* LEGO: check in for Localization to temporary branch. (microsoft#745)

Co-authored-by: csigs <[email protected]>

* LEGO: check in for Localization to temporary branch. (microsoft#746)

Co-authored-by: csigs <[email protected]>

* LEGO: check in for Localization to temporary branch. (microsoft#747)

Co-authored-by: csigs <[email protected]>

* LEGO: check in for Localization to temporary branch. (microsoft#748)

Co-authored-by: csigs <[email protected]>

* Localized file check-in by OneLocBuild Task: Build definition ID 10947: Build ID 14905562

Localized file check-in by OneLocBuild Task

* Returned back original names

* Removed redundant locale - test

* Removed redundant folders

* Localized file check-in by OneLocBuild Task: Build definition ID 10947: Build ID 14906155

Localized file check-in by OneLocBuild Task

* Returned back changes. Removed redundant

* Create PR in OneLocBuild task only on third week of sprint (microsoft#755)

* Fix localization pipeline

* Add missed change

* Added option to disable PR creation

* Add OneLocBuild removal to the localization pipeline (microsoft#804)

* Add OneLocBuild removal to the pipeline

* fix ignore error to double if

Co-authored-by: Ilya Kuleshov <[email protected]>

* Localization update (microsoft#802)

* Removing Localize folder

* Revert "Removing Localize folder"

Co-authored-by: Ilya Kuleshov <[email protected]>

* Removed OneLocBuild folder

* Move notifications about Task-lib Localization PR from Slack to MS Teams - Part 1 (microsoft#816)

* Localized file check-in by OneLocBuild Task: Build definition ID 10947: Build ID 18673037 (microsoft#879)

* Juno: check in to lego/hb_a4aa9cc4-603b-418e-91f1-700184175625_20221105085105779. (microsoft#881)

* Juno: check in to lego/hb_a4aa9cc4-603b-418e-91f1-700184175625_20221106085044954. (microsoft#882)

* Localized file check-in by OneLocBuild Task: Build definition ID 10947: Build ID 18697074 (microsoft#883)

* Update LocProject file

* Update LocProject

* Remove lowercase loc strings on windows

* Removing Localize and OneLocBuild folder

Co-authored-by: Anatolii Bolshakov (Akvelon INC) <[email protected]>
Co-authored-by: csigs <[email protected]>
Co-authored-by: Egor Bryzgalov <[email protected]>
Co-authored-by: Anatoly Bolshakov <[email protected]>
Co-authored-by: csigs <[email protected]>
Co-authored-by: Nikita Ezzhev <[email protected]>
Co-authored-by: kuleshovilya <[email protected]>
Co-authored-by: Ilya Kuleshov <[email protected]>
Co-authored-by: Denis Tikhomirov <[email protected]>
Co-authored-by: Maksim Petrov <[email protected]>
* Pipeline fixing

* Indentation fix

* Formating

* Node version was reverted
* Migrating to Node 16 (microsoft#844)

* create new release/4.x branch

* release for 4 version

* update pipeline

* update package.json

* fix test and build commands

* fixed tests on node18

* change image version for linux

* fix test for linux

* add separete test for linux

* node v16.13

* add v16.13.0 to test

* test on windows-2022, macOS-11

* separate branch for node16

* remove temporary code

* update package.json

* changed pipeline

* updated pipeline

* Microsoft mandatory file (microsoft#839)

* Update pipeline to include a step for publishing artifacts. (microsoft#848) (microsoft#849)

* Update pipeline

* Update azure-pipelines.yml

* nodeversion as variable

* Added node16 to schema (microsoft#852)

Co-authored-by: Andrey Ivanov <[email protected]>

* bump version (microsoft#862)

Co-authored-by: Andrey Ivanov <[email protected]>

* Update ci-cd for publishing npm package (microsoft#863)

* fix npm publish

Co-authored-by: Andrey Ivanov <[email protected]>

* fix add registry (microsoft#864)

Co-authored-by: Andrey Ivanov <[email protected]>

* fix ToolRunner - _getSpawnSyncOptions (microsoft#873)

* Mockery version is updated from 1.7.0 to 2.1.0 because of vulnerability (microsoft#878)

* Fixed release path (microsoft#888)

* Improvement of the release condition (microsoft#889)

* Added test task with test condition. Negative test

* Added test task with test condition. Positive test

* Fix path

* Fix path

* Fix path

* Changed build.sourcebranchname -> build.sourcebranch and path to branch

* Fix path

* Version changed (microsoft#894)

* Pipeline fixing (microsoft#893)

* Token test

* Test token

* Publish test

* call order changed

* Nmp token syntax reverted

* Changed windows job

* Script clearing

* Uncommented condition

* Token changed

* Test - off publish condition

* Switched on publish condition

* Package version was reverted

* Add escape for more correct working

* Add changelog

* Update package-lock version

* Bump tl version to 4.1.0

* Update changelog

* Remove PS task lib changes from node changelog

* Update changelog for 4.0.1-preview

Co-authored-by: AndreyIvanov42 <[email protected]>
Co-authored-by: Andrey Ivanov <[email protected]>
Co-authored-by: Denis Rumyantsev <[email protected]>
Co-authored-by: Roman-Shchukin <[email protected]>
The event "unhandledRejection" was added on process.on because when Promise is rejected and there is no catch
the error will not be thrown and process.on('uncaughtException') won't call for Node10 but it will emit "uncaughtException" for node 16.
So we need to listen to "unhandledRejection" and throw the error to make sure the error is thrown and the result will be similar.
* Improve types for argIf command

* Bump package version

* Update changelog
In case the command line arguments contain secrets or any other info that should not be printed out. We could use the same 'silent' flag to not add all the arguments to debug log.
@andrelvahter andrelvahter requested review from a team as code owners March 14, 2023 07:23
@andrelvahter
Copy link
Author

@microsoft-github-policy-service agree company="Microsoft"

@microsoft-github-policy-service agree company="Microsoft"

Move bit down, so the options are available
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.