Azure / logicapps

Azure Logic Apps labs, samples, and tools
MIT License
367 stars 302 forks source link

Actions with managed connectors described as app settings parameters are not properly visible in LA Standard Designer #463

Closed WenovateAA closed 2 years ago

WenovateAA commented 2 years ago

Hi,

Since yesterday I see error in designer for actions using managed connectors: The provided subscription identifier '@appsetting('WORKFLOWS_SUBSCRIPTION_ID')' is malformed or invalid. image Connectors are configured using app setting parameters:

"azuretables": {
            "api": {
                "id": "/subscriptions/@appsetting('WORKFLOWS_SUBSCRIPTION_ID')/providers/Microsoft.Web/locations/westeurope/managedApis/azuretables"
            },
            "authentication": {
                "type": "ManagedServiceIdentity"
            },
            "connection": {
                "id": "/subscriptions/@appsetting('WORKFLOWS_SUBSCRIPTION_ID')/resourceGroups/@appsetting('WORKFLOWS_RESOURCE_GROUP_NAME')/providers/Microsoft.Web/connections/azuretables"
            },
            "connectionRuntimeUrl": "@appsetting('AZURETABLES_CONNECTION_RUNTIMEURL')"
        }

Similar problem can be seen in workflow's run history: image

This is a way described in CI/CD example of LA dev sample: https://github.com/Azure/logicapps/tree/master/azure-devops-sample It doesn't prevent a workflow from running, somehow designer can't treat it properly. Problem was first detected yesterday, after some time dissapeared, now I see it again. Connector configuration is active since several months and problem are observed only now.

AB#16678652

rob-thijssen commented 2 years ago

Exact same behaviour on my side! Please provide the fix asap. It took me all day up-to-now to find out if the error was somehow invoked by me with a wrong deployment or so. Thanks to this post I found out that the logicapp run will not (yet) be influenced. I even tried to add { } to the appsetting as the new documentation (here). No difference. Error still visible in designer. As an addition to this issue: The connections overview also displays errors: image There are correct access policies deployed on the managed api connection.

image with { }: image

motterasysco commented 2 years ago

Same issue with me as well. I discovered this today.

hartra344 commented 2 years ago

We have figured out the cause and have started to deploy a fix. Due to how our deployments happen as a rollout this may take a few days to fully reach production.

In the meantime here is a link that will make sure the fixed bits load if this is something that is blocking your workflow: https://aka.ms/logicAppActionFix

motterasysco commented 2 years ago

Still the same issue (northeurope). Can you explan the fix you provides here? Does not do much..

We have figured out the cause and have started to deploy a fix. Due to how our deployments happen as a rollout this may take a few days to fully reach production.

In the meantime here is a link that will make sure the fixed bits load if this is something that is blocking your workflow: https://aka.ms/logicAppActionFix

rob-thijssen commented 2 years ago

Follow-up: I have been monitoring our logicapps last two weeks and this issue seems to be resolved for West Europe.

WenovateAA commented 2 years ago

Same problem is happening again @shailesh-agre

rob-thijssen commented 2 years ago

Same problem is also happening again here @hartra344 . Region: West Europe

krisvijaykb commented 2 years ago

Same issue. Any updates on this?

krisvijaykb commented 2 years ago

The issue got resolved automatically for me.

bizintelligence commented 2 years ago

Hi, I have been facing the same issue. However, I have noted that if I use curly bracket in local development environment then it works locally but once it is deployed to Logic App then I see "malformed" message.

Therefore, I tried to remove the curly bracket before publishing it to remote repo. It works on Logic App.

Before { "managedApiConnections": { "office365": { "api": { "id": "/subscriptions/@{appsetting('WORKFLOWS_SUBSCRIPTION_ID')}/providers/Microsoft.Web/locations/australiasoutheast/managedApis/office365" }, "connection": { "id": "/subscriptions/@{appsetting('WORKFLOWS_SUBSCRIPTION_ID')}/resourceGroups/@{appsetting('WORKFLOWS_RESOURCE_GROUP_NAME')}/providers/Microsoft.Web/connections/office365" }, "connectionRuntimeUrl": "@{appsetting('OFFICE365_CONNECTION_RUNTIMEURL')}", "authentication": { "type": "ManagedServiceIdentity" } } } } After { "managedApiConnections": { "office365": { "api": { "id": "/subscriptions/@appsetting('WORKFLOWS_SUBSCRIPTION_ID')/providers/Microsoft.Web/locations/australiasoutheast/managedApis/office365" }, "connection": { "id": "/subscriptions/@appsetting('WORKFLOWS_SUBSCRIPTION_ID')/resourceGroups/@appsetting('WORKFLOWS_RESOURCE_GROUP_NAME')/providers/Microsoft.Web/connections/office365" }, "connectionRuntimeUrl": "@appsetting('OFFICE365_CONNECTION_RUNTIMEURL')", "authentication": { "type": "ManagedServiceIdentity" } } } }

However, this is so much pain. There has to be fix for it.

AbodeSaafan commented 2 years ago

Hello,

There was a bug that we hotfixed that solved the issue mentioned earlier in this thread.

@bizintelligence We are aware of this issue. Basically, right now, if you are developing with the designer on the portal, you must use the appsetting/parameter expression in connections.json without the proper string interpolation format, so, without the wrapping { }.

We are working on a plan to bring the support to use the proper string interpolated format to the portal designer. When we have that, we will share it with you and other customers and clearly outline what is supported.

Let me know if you have any other questions here.

nikhilbattula9 commented 1 year ago

My logic app is in switzerland north region .I got the same error tried using both ways without using curly braces @appsetting('subscription_id') and with curly braces @{appsetting('subscription_id')} . app setting parameter values is'nt reflecting into the connections.json file. Could u guys pls let me know if you found any work around (except creating mutliple files for each environment)