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

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

Open
chasers opened this issue Feb 8, 2024 · 4 comments

Comments

@chasers
Copy link
Contributor

chasers commented Feb 8, 2024

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
Copy link
Contributor Author

chasers commented Feb 9, 2024

Can't replicate:

Image

The log for bad password:

ClientHandler: Exchange error: "Wrong password" when method :auth_query

@chasers
Copy link
Contributor Author

chasers commented Feb 9, 2024

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
Copy link
Contributor Author

chasers commented Feb 9, 2024

Closing for now...

@chasers chasers closed this as completed Feb 9, 2024
@chasers chasers changed the title Better error when bad password is submitted Better error when there is no . in the username or tenant reference in the options params Feb 9, 2024
@chasers
Copy link
Contributor Author

chasers commented Feb 9, 2024

Updated and re-opened

@chasers chasers reopened this Feb 9, 2024
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

No branches or pull requests

1 participant