fix(frontend): support special chars in postgresql client [#1775]

[mrl5]

Fix #1775

tested with:

1. setup fresh postgres15.3 container with POSTGRES_PASSWORD kot@behemot#zonk
2. get DB URI with

const args = { user: 'postgres', password: 'kot@behemot#zonk', host: '0.0.0.0', dbname: 'postgres', sslmode: 'disable' };
const u = new URL(`postgres://${args.user}:${args.password}@${args.host}:${args.port ?? 5432}/${args.dbname}?sslmode=${args.sslmode}`);
u.hash = '';
u.search = `?sslmode=${args.sslmode}`;
u.pathname = args.dbname;
u.host = args.host;
u.password = args.password;
u.username = args.user;

u.toString();

3. connect to db

psql 'postgres://postgres:kot%40behemot%23zonk@0.0.0.0/postgres?sslmode=disable'

[mrl5]

I also checked if there is something similar for mysql or mongo but didn't found anything similar to modified code

[rubenfiszel]

Thanks for the PR, I just reworked that part and the change need to be applied to TestConnection.svelte

Side-note: I'm working on adding mysql as a first class language too :)

[mrl5]

I wonder if there's anything that's stopping us with using ClientOptions here?

The only reason that I came up with was some backwards compatibility with args interface. Simple adapter like this could work:

-import { Client } from 'https://deno.land/x/postgres/mod.ts'
+import { Client, ClientOptions } from 'https://deno.land/x/postgres/mod.ts';
 export async function main(args: any) {
-       const client = new Client("postgres://" + args.user + ":" + args.password + "@" + args.host + ":" + (args.port ?? 5432) + "/" + args.dbname + "?sslmode=" + args.sslmode)
+       const clientOptions: ClientOptions = {
+               database: args.dbname,
+               port: args.port ?? 5432,
+               hostname: args.host,
+               password: args.password,
+               user: args.user,
+       };
+       const client = new Client(clientOptions)
        await client.connect()
        return 'Connection successful'
 }

however there would be issues with mapping TLSOptions to psql sslmode - notice lack of args.sslmode in proposed adapter