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

Add default user and show warning message on startup when no users found #2481

Merged

Conversation

jfcalvo
Copy link
Member

@jfcalvo jfcalvo commented Mar 6, 2023

With this PR we will add a default argilla user (with argilla) when there are not users on the database.

We also show a warning message indicating that it could be dangerous to have default argilla user in a production environment and what to do in that case.

In the case that we already have users on the database (because of the migration) we check if the argilla default user is using default credentials (password and api key) and in that case we show the same previous warning.

@jfcalvo jfcalvo requested a review from frascuchon March 6, 2023 11:06
@jfcalvo jfcalvo merged commit 68295ce into feat/add-basic-database-support Mar 6, 2023
@jfcalvo jfcalvo deleted the add-default-user-on-startup branch March 6, 2023 14:18
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