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

fix: incorrect connection string #4168

Closed
wants to merge 2 commits into from

Conversation

lizhineng
Copy link
Contributor

The valid database connection string should be in the format of:

postgresql://[user[:password]@][[host][:port]][/dbname]

@lizhineng lizhineng requested a review from a team as a code owner September 3, 2024 02:38
@lizhineng lizhineng changed the title fix(docker): incorrect connection string fix: incorrect connection string Sep 3, 2024
gary02
gary02 previously approved these changes Sep 5, 2024
@gary02
Copy link
Contributor

gary02 commented Sep 5, 2024

Thanks for your pull request @lizhineng .

We just updated develop branch to allow fork PRs to run Github Actions, please rebase the branchs so that we can merge your approved PRs.

@lizhineng lizhineng closed this Dec 10, 2024
@gitwoz
Copy link
Contributor

gitwoz commented Feb 18, 2025

@lizhineng Sorry we missed this PR, however, we have already fixed it at ea972c1, thanks for your contribution!

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

Successfully merging this pull request may close these issues.

3 participants