fix(pg-v5): Fix databasename for URLs with query element and be more consistent #1882
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
For operations such as pg:psql, we have been assuming that the URL parsed
path
method is enough to find the database name. This is incorrect when we pass parameters such assslmode
. In general, this problem has been solved withparsePostgresConnectionString
but not everything uses it.This fix removes any custom parsing we are doing of URLs and moves to using this function everywhere to avoid inconsistency.
Without this,
pg:psql
was failing on verify-full URL's but with this patch, it works fine.