supabase / supavisor

A cloud-native, multi-tenant Postgres connection pooler.
https://supabase.github.io/supavisor/
Apache License 2.0
1.66k stars 55 forks source link

Better error when there is no `.` in the username or tenant `reference` in the `options` params #302

Open chasers opened 5 months ago

chasers commented 5 months ago

Respond with a good error message when there is no . in the username to split on to pull the tenant from and there are no options params to pull the tenant from either.

chasers commented 5 months ago

Can't replicate:

Image

The log for bad password:

ClientHandler: Exchange error: "Wrong password" when method :auth_query
chasers commented 5 months ago

And when I try a user which does not exist:

Image

And the log message:

ClientHandler: Authentication auth_secrets error: "There is no user 'joe' in the database. Please create it or change the user in the config"
chasers commented 5 months ago

Closing for now...

chasers commented 5 months ago

Updated and re-opened