Skip to content

Conversation

@pkhartsk
Copy link
Contributor

URI encode username and database name, put password in quotes

in order to not have to include it in the database name, which it shouldn't be a part of
Copy link
Member

@phracek phracek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please address my issues. The rest is fine.

@pkhartsk
Copy link
Contributor Author

Resolves #154 as well now

@pkhartsk pkhartsk force-pushed the postgresql_cmd_improvements branch from 9eaecbc to 46ae0ab Compare January 15, 2026 13:45
Copy link
Member

@phracek phracek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@phracek phracek merged commit e43d18e into sclorg:main Jan 23, 2026
4 checks passed
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.

2 participants