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

Implement support for token authentication in MSDeploy #19130

Merged
merged 4 commits into from
Oct 17, 2023

Conversation

onetocny
Copy link
Contributor

@onetocny onetocny commented Oct 17, 2023

Task names:

  • AzureRmWebAppDeploymentV3
  • AzureRmWebAppDeploymentV4

Description: Use latest azure-pipelines-tasks-webdeployment-common package in order to leverage its support of token authentication in MSDeploy.

Documentation changes required: N

Added unit tests: Y

Attached related issue: link

Checklist:

  • Task version was bumped - please check instruction how to do it
  • Checked that applied changes work as expected

@onetocny
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 4 pipeline(s).

@onetocny onetocny changed the title Use latest azure-pipelines-tasks-webdeployment-common in AzureRmWebAppDeploymentV4 Implement support for token authentication in MSDeploy Oct 17, 2023
@onetocny onetocny marked this pull request as ready for review October 17, 2023 12:27
@onetocny onetocny requested a review from rvairavelu as a code owner October 17, 2023 12:27
@onetocny onetocny merged commit 8de437d into master Oct 17, 2023
8 checks passed
@onetocny onetocny deleted the users/onetocny/msdeploy_token_auth branch October 17, 2023 13:02
DmitriiBobreshev added a commit that referenced this pull request Oct 18, 2023
DmitriiBobreshev added a commit that referenced this pull request Oct 18, 2023
* [AzureRmWebAppDeploymentV4] Migrate task to Node16

- task migration to node 16

* [AzureRmWebAppDeploymentv4] Node16 migration

- Added default version for AzureRmWebAppDeploymentv4

* [AzureRmWebAppDeploymentv4] Node16 migration

-- Added changes from the PR #19130
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.

3 participants