Restored WorkloadIdentityFederation support

[starkmsu]

Task name: AzureFunctionAppV1, AzureFunctionAppV2

Description: Restored WorkloadIdentityFederation support

Documentation changes required: (Y/N) N

Added unit tests: (Y/N) N

Attached related issue: (Y/N) N

Checklist:

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

[PavloAndriiesh]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 4 pipeline(s).

[FinVamp1]

Hello,

While in general I understand using Azure-Arm-Rest-v2 helps avoid duplication of code it does mean that we have lost the targeted logging that differentiates Functions from App Service. I note that you have removed the Functions flavour of the App Service and Kudu classes which means that we lose the targeted logging for Functions,. console.log("##vso[telemetry.publish area=TaskDeploymentMethod;feature=AzureFunctionAppDeployment]" + authMethodtelemetry); . This may seem like a nit but we are trying to distinguish between the tasks so we can look into the telemetry and dientify failures before the task gets to us? Perhaps we can pass in parameter to the common logic?

[FinVamp1]

This looks great. I think having the FileTransformationUtility as part of the shared packages is the right thing to do.