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

Fail to delete the database of the attached PostgreSQL server #2259

Open
v-xinda opened this issue Mar 8, 2024 · 1 comment
Open

Fail to delete the database of the attached PostgreSQL server #2259

v-xinda opened this issue Mar 8, 2024 · 1 comment
Labels
AT-CTI Created by CTI team

Comments

@v-xinda
Copy link

v-xinda commented Mar 8, 2024

OS: All
Build Version: 20240306.1
Regression: Not a regression

Repro Steps:

  1. Create a PostgreSQLSingle server -> Expand it.
  2. Right click the database "postgres" -> Click "Copy Connection String...".
  3. Attach PostgreSQLSingle server without database name, like postgres://us:Daxinge0104@anicole-f-d.postgres.database.azure.com:5432?ssl=true.
  4. Navigate to Workspaces -> Right click the attached PostgreSQLSingle server -> Create a new database.
  5. Right click the above database -> Delete Database... -> Delete.
  6. Chech whether succeeds to delete the database.

Expect:
Succeed to delete the database.

Actual:
Fail to delete the database.
image

More Info:

  1. This issue also reproduces for the PostgreSQLFlexible server.
  2. This issue also reproduces for the unattached PostgreSQL server.
@v-xinda v-xinda added the AT-CTI Created by CTI team label Mar 8, 2024
@JasonYeMSFT
Copy link
Contributor

An open connection to the database would prevent people from deleting it. The user can mitigate this issue by connecting to another database. The extension should have a way to reset connection because that database could be the last one that the user can connect to from the extension because we don't let people connect to default databases from the UI.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AT-CTI Created by CTI team
Projects
None yet
Development

No branches or pull requests

2 participants