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

Selecting server/database from Azure tree does not pick up correct account #21643

Closed
cheenamalhotra opened this issue Jan 19, 2023 · 2 comments · Fixed by #21749
Closed

Selecting server/database from Azure tree does not pick up correct account #21643

cheenamalhotra opened this issue Jan 19, 2023 · 2 comments · Fixed by #21749

Comments

@cheenamalhotra
Copy link
Member

cheenamalhotra commented Jan 19, 2023

  • Azure Data Studio Version: RC1
  • OS Version: Windows 11

Steps to Reproduce:

  1. In Azure tree, connect to multiple accounts using ADAL
  2. Migrate accounts to MSAL (update auth library and reload)
  3. Reauthenticate to accounts as prompted.
  4. Connect to any resource from second+ account in list.
  5. The account loaded in dropdown is the first one in the list and not the one associated with the resource.

This is a new issue, and root cause is likely related to MSAL <> ADAL account id comparison when loading associated account in dropdown, as account id is different for ADAL live# accounts, and is not matched with MSAL live# accounts, but needs further investigation.

The issue does not apply to newly added MSAL accounts, and only applies to migrated accounts.

@cheenamalhotra
Copy link
Member Author

cheenamalhotra commented Jan 20, 2023

Just discussed with @cssuh, it seems this issue is not consistently reproducible, and doesn't happen with ADAL accounts all the time. The steps above are not reproing the issue again in RC build, we've tried multiple rounds, so it does lead us to think about lowering the urgency of this issue until we find a consistent repro to be able to confirm the fix we make is accurate.

@kenvanhyning @kburtram if you agree, we'd like to move it to March, and will keep close eye on possibilities that would lead to this issue when it happens again.

@cheenamalhotra cheenamalhotra added Triage: Postponed Items that were planned for a release but didn't make it for one reason or another. Pri: 2 and removed Approved Triage: Done Pri: 0 labels Jan 20, 2023
@cssuh
Copy link
Member

cssuh commented Jan 20, 2023

To add more context, I thought this was referring to the account dropdown in the firewall rule dialog screen, so that is what I was working on (the firewall rule account selected by default did not match). I can still make the PR but for that issue it doesn't need to go into the Jan release.

@kburtram kburtram removed the Triage: Postponed Items that were planned for a release but didn't make it for one reason or another. label Jan 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants