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

Azure Resource Manager: While creating new connection showing a error message #1251

Open
vgouth opened this issue Nov 13, 2024 · 16 comments
Open

Comments

@vgouth
Copy link

vgouth commented Nov 13, 2024

Describe the Bug with repro steps

  1. Create a consumption logic app
  2. Open designer and add http request trigger
  3. Add any action from Azure Resource Manager Eg: Read a subscription
  4. Create a new connection, if there is any existing connection click on change connection and select Add new
  5. Change Authentication type to Service Principal
  6. Give connection name as mandatory field
  7. Click on create new
  8. Connection status is showing some error

Expected: Connection status should show connected.

Actual: Connection status is showing error message.

What type of Logic App Is this happening in?

Consumption (Portal)

Which operating system are you using?

Windows

Are you using new designer or old designer

New Designer

Did you refer to the TSG before filing this issue? https://aka.ms/lauxtsg

Yes

Workflow JSON

Screenshots or Videos

Image

Image

Browser

Edge

Additional context

Version : 2.41028.1.6

@ccastrotrejo ccastrotrejo self-assigned this Nov 14, 2024
@ccastrotrejo
Copy link

Talked with @vgouth offline and the connection is created successfully, however as there is clientSecret and clientID when creating it. This is expected by design.

@ccastrotrejo ccastrotrejo closed this as not planned Won't fix, can't repro, duplicate, stale Nov 14, 2024
@ccastrotrejo ccastrotrejo reopened this Nov 15, 2024
@ccastrotrejo
Copy link

Reopen it and marked it as a feature request to mark the secret and id as required

@vgouth
Copy link
Author

vgouth commented Nov 22, 2024

Issue still repro in latest test pass
Client secret and client id should be in mandatory field if those fields are required for creating a connection.
Version: 0.2022.1.794

Image

@ccastrotrejo ccastrotrejo transferred this issue from Azure/LogicAppsUX Nov 26, 2024
@ccastrotrejo ccastrotrejo removed their assignment Dec 6, 2024
@ccastrotrejo
Copy link

The API call to get the connector properties retrieve the inputs as not required

@vgouth
Copy link
Author

vgouth commented Dec 18, 2024

Issue repro in latest canary environment
Version: 0.2022.1.806
Image

@vgouth
Copy link
Author

vgouth commented Dec 26, 2024

Issue repro in latest canary test pass
Version: 0.2022.1.811

1 similar comment
@vgouth
Copy link
Author

vgouth commented Jan 6, 2025

Issue repro in latest canary test pass
Version: 0.2022.1.811

@vgouth
Copy link
Author

vgouth commented Jan 10, 2025

Issue repro in latest staging test pass

Version: 2.41231.2.4
Image

@takyyon
Copy link

takyyon commented Jan 10, 2025

@ccastrotrejo Mind looking into this please?

@ccastrotrejo
Copy link

Hi @takyyon this is something that backend engineers need to take a look at. The reason on why we are displaying the error message after creating the connection, is because its getting created without Client ID and Client secret. Therefore there is no way to get a solid connection without these.

I checked the connector and neither of these properties come back as required. They need to enable that.

@takyyon
Copy link

takyyon commented Jan 10, 2025

@ccastrotrejo Do you mind starting a thread with backend teams for this? I'll consider this as a non-blocker for our deployment right now but let me know if you feel otherwise.

@ccastrotrejo
Copy link

@takyyon Yep I can do that, and I agree, this isn't a blocker for the deployment

@vgouth
Copy link
Author

vgouth commented Jan 28, 2025

Issue repro in latest test pass
Version: 2.50107.1.3

Image

@vgouth
Copy link
Author

vgouth commented Feb 4, 2025

Issue repro in latest stage test pass
Version: 2.50203.1.1
Image

@vgouth
Copy link
Author

vgouth commented Feb 11, 2025

Issue repro in latest canary test pass
Version: 2.0.20250207.1

@vgouth
Copy link
Author

vgouth commented Mar 3, 2025

Issue repro in latest stage test pass
Version: 2.50224.1.3

Image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants