AzureWebAppContainerV1: AppSettings updated after App Service restart

[sergeyzwezdin]

Type: Bug
Enter Task Name: AzureWebAppContainerV1

list here (V# not needed): AzureWebAppContainer
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks

Environment

• Server - Azure Pipelines
• Agent - Hosted

Issue Description

While deploying test container to Azure App Services, I noticed that the service restarted right after deployment and then app settings changed. It seems that's wrong because of in this case App Service sometimes fails with error that Docker credentials are not found.

What if put these lines before calling deployWebAppImage?

azure-pipelines-tasks/Tasks/AzureWebAppContainerV1/azurermwebappdeploymentprovider.ts

Lines 42 to 45 in 46cca41

	if(this.taskParams.AppSettings) {
	var customApplicationSettings = ParameterParser.parse(this.taskParams.AppSettings);
	await this.appServiceUtility.updateAndMonitorAppSettings(customApplicationSettings);
	}

[AmrutaKawade]

This is fixed and will be deployed within 2 weeks