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

An error occurs after attaching "postgres" database for a PostgreSQL account #2087

Open
v-chenzha opened this issue Dec 21, 2022 · 5 comments
Assignees
Labels
AT-CTI Created by CTI team bug The issue reveals something not working as expected. postgres
Milestone

Comments

@v-chenzha
Copy link

OS: Win10
Build Version: 20221220.1

Repro Steps:

  1. Create a PostgreSQL account.
  2. Expand PostgreSQL account -> Right click "postgres" -> Copy Connection String.
  3. Attach "postgres" database with connection string?ssl=true.
  4. Check whether no error occurs after attaching "postgres" database.

Expect:
No error occurs after attaching "postgres" database.
image

Actual:
An error occurs after attaching "postgres" database.
image

More Info:
There is an extra single quote in the Connect String. No error occurs after deleting the single quotes, such as:
error connection string: postgres://cindy%40cindy-single:'Tcd...815615'@cindy-single.postgres.database.azure.com:5432/postgres
correct connection string: postgres://cindy%40cindy-single:Tcd...815615@cindy-single.postgres.database.azure.com:5432/postgres

@v-chenzha v-chenzha added the AT-CTI Created by CTI team label Dec 21, 2022
@MicroFish91 MicroFish91 added the bug The issue reveals something not working as expected. label Feb 2, 2023
@MicroFish91 MicroFish91 added this to the 0.19.0 milestone Feb 2, 2023
@v-ruizh
Copy link

v-ruizh commented Aug 24, 2023

Verified on the build 20230822.5, the below error occurs for the attached "PostgreSQL server".
image

@JasonYeMSFT JasonYeMSFT self-assigned this Sep 11, 2023
@JasonYeMSFT JasonYeMSFT modified the milestones: 0.19.0, 0.20.0 Sep 11, 2023
@JasonYeMSFT
Copy link
Contributor

@v-ruizh I made a fix for this bug.

@v-ruizh
Copy link

v-ruizh commented Sep 12, 2023

Verified this issue on the extension build 20230911.4, another error occurs when attaching PostgreSQL server with the copied connection string.
image

The extra single quote still shows in the copied Connect String. Delete the extra single quote and add "?ssl=true" to the connection string, the error disappears.

@JasonYeMSFT
Copy link
Contributor

I believe this has been fixed. If this still reproduces, please re-open this issue.

@v-ruizh
Copy link

v-ruizh commented Oct 18, 2023

Verified this bug on the latest build 20231017.2, it still reproduces. Reopen it.

@v-ruizh v-ruizh reopened this Oct 18, 2023
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 bug The issue reveals something not working as expected. postgres
Projects
None yet
Development

No branches or pull requests

5 participants