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

Adding multiple similar connections doesn't work correctly #18001

Open
kburtram opened this issue Aug 27, 2024 · 4 comments
Open

Adding multiple similar connections doesn't work correctly #18001

kburtram opened this issue Aug 27, 2024 · 4 comments

Comments

@kburtram
Copy link
Member

kburtram commented Aug 27, 2024

Steps to Reproduce:

  1. Using the connection dialog, create a saved connection to a database on a server using SqlLogin
  2. Using the connection dialog, create a saved connection to the server using Entra ID

Expected: Two distinct connections, one to DB and one to Server with different auth types. Clearly distinguishable in connection display name. Note: I'm not sure the repro needs to be this specific, probably any slightly ambiguous connections will have similar problems.

Actual: Two connections have same title, one of them clearly wrong. The contents in the below screenshot show both a DB and Server type connection (based on the subnodes), but the display names are identical. The second one being incorrect.

image

@kburtram
Copy link
Member Author

This is similar repro when adding a connection from the MRU. I had two entries added to settings JSON for same the connection, one with only minimal details. They both show in Connections list but can't tell them apart.

Also, I don't think those generated profileNames should be put into the settings file since they are supposed to be auto-generated based on connection metadata. For example, if the metadata changes will the saved name be overridden with the new data & how can we tell the difference between the auto-gen name and user provided "friendly name"?

image

@Benjin Benjin self-assigned this Sep 4, 2024
@Benjin
Copy link
Contributor

Benjin commented Sep 9, 2024

I don't think those generated profileNames should be put into the settings file since they are supposed to be auto-generated based on connection metadata

Agreed; fixed as part of #18030

@Benjin
Copy link
Contributor

Benjin commented Sep 9, 2024

Though I've seen this issue before, I haven't been able to repro it now that I'm digging into it. Can you try recording the exact steps you use to trigger this bug?

Win + Shift + R will trigger Windows' built-in screen recording tool for the easiest way to do that.

@Benjin
Copy link
Contributor

Benjin commented Feb 20, 2025

This should be mostly resolved with #18684, though I still need to close some corner cases, like ensuring the displayname is differentiated.

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

2 participants